Ticket #7654 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Print Mantid version number

Reported by: Vickie Lynch Owned by: Andrei Savici
Priority: major Milestone: Release 3.0
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description

SNS user requests that the Welcome to Mantid line that appears running a python script include the version of Mantid running. He accidently had both the release and the nightly build in his PYTHONPATH and could not get new options for an algorithm to work.

Change History

comment:1 Changed 7 years ago by Andrei Savici

  • Owner set to Andrei Savici
  • Status changed from new to inprogress
  • Milestone changed from Backlog to Release 2.6.1

comment:2 Changed 7 years ago by Andrei Savici

Verbose welcome message. Refs #7654.

The SettingsTest.py is python api version 1. In version 2, the welcomeMessage function is no longer exposed to python

Changeset: 6d7af9a88314a0d5ad1433e424777d2f75095f89

comment:3 Changed 7 years ago by Andrei Savici

Simplified version display. Refs #7654

Changeset: 85bd93d994318efaaec669e0b3727bc80c078350

comment:4 Changed 7 years ago by Andrei Savici

Removed api v1 test. Refs #7654

Changeset: e715428719b7d094707a52bcdc9c54dcebff7c71

comment:5 Changed 7 years ago by Andrei Savici

  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:6 Changed 7 years ago by Andrei Savici

Remove test from Cmake. Refs #7654

Changeset: 9c31f22657d8d5611d833a033b8484fce55ed5cc

comment:7 Changed 7 years ago by Russell Taylor

  • Milestone changed from Release 2.6.1 to Release 3.0

comment:8 Changed 7 years ago by Russell Taylor

N.B. There is an existing log message at 'information' level that gives the Mantid version and revision when importing Mantid in a python script.

comment:9 Changed 7 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:10 Changed 7 years ago by Russell Taylor

  • Status changed from verifying to reopened
  • Resolution fixed deleted

A crucial "-" between "Mantid" and "Manipulation" has gone missing. Also, there's a blank line appeared after the message in MantidPlot.

All will be well if you make your welcomeMessage function look like this:

return "Welcome to Mantid " + std::string(Mantid::Kernel::MantidVersion::version()) + " - Manipulation and Analysis Toolkit for Instrument Data";

comment:11 Changed 7 years ago by Andrei Savici

  • Status changed from reopened to inprogress

Update the welcome string. Refs #7654

Changeset: dc8e839ac1f2e77268809bf6d9acf3c10667d1e9

comment:12 Changed 7 years ago by Andrei Savici

  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:13 Changed 7 years ago by Russell Taylor

  • Status changed from verify to verifying

comment:14 Changed 7 years ago by Russell Taylor

  • Status changed from verifying to closed

The addition is a bit redundant in some ways, but it's hardly intrusive so I think it's OK. The removal of the test it OK given the very imminent demise of the v1 Python API.

comment:15 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8499

Note: See TracTickets for help on using tickets.