Ticket #10162 (closed: fixed)
TofConverter gui is broken.
Reported by: | Harry Jeffery | Owned by: | Harry Jeffery |
---|---|---|---|
Priority: | critical | Milestone: | Release 3.3 |
Component: | GUI | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Michael Reuter |
Description
Open Interfaces->Utility->TofConverter
File "/home/harry/MantidProject/main/repo/Code/Mantid/scripts/TofConverter/converterGUI.py", line 70 else: ^ SyntaxError: invalid syntax at line 1 in '<Interface>'
Change History
comment:3 Changed 6 years ago by Harry Jeffery
- Status changed from assigned to verify
- Resolution set to fixed
comment:4 Changed 6 years ago by Harry Jeffery
- Status changed from verify to reopened
- Resolution fixed deleted
Let's fix the tabs while we're at it.
comment:5 Changed 6 years ago by Harry Jeffery
- Status changed from reopened to inprogress
Fix tabs and other issues in convertGUI.py
Other issues fixed:
- pi was hardcoded and never used
- e2cm had been mistyped
- blank line usage was inconsistent
Refs #10162.
Changeset: 6f3c4a65aeea1a658a8ce196451d33e3583b9ab0
comment:6 Changed 6 years ago by Harry Jeffery
Fix error when converting to momentum transfer.
The gui would complain that theta needs to be > 0 when a value had been entered correctly.
Refs #10162.
Changeset: 9ffc16c84c46741e19518ed7e500940aabfea6bd
comment:7 Changed 6 years ago by Harry Jeffery
- Status changed from inprogress to verify
- Resolution set to fixed
TofConverter should be much happier now.
Testing
- Open Interfaces->Utility->TofConverter
- Play about with the unit conversions
- Verify conversion to momentum transfer works for values of theta (scattering angle) > 0.
- Code inspection
- Use git diff -w to see all non-whitespace changes.
comment:8 Changed 6 years ago by Michael Reuter
- Status changed from verify to verifying
- Tester set to Michael Reuter
comment:9 Changed 6 years ago by Michael Reuter
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/bugfix/10162_tofconverter_gui_is_broken'
Full changeset: 06e1a705eb31ccca5a9dd891a1baf8f255bd952d
comment:10 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 11004
Fix broken TofConverter gui.
Refs #10162.
Changeset: 7684e001e8da1ea6f886db2c4cd58ff81108bec6