Ticket #7654 (closed: fixed)
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: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: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
