Ticket #1747 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

SANS - file saving, minor issues

Reported by: Nick Draper Owned by: Steve Williams
Priority: major Milestone: Iteration 26
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Anders Markvardsen

Description

Should we have default extensions on the files for “save result”, at least for type “RKH” it saves the file with no extension on the name (needs either .Q or .txt) so I have to remember to type it in to the filename box every time.

Yes. It’s important that the cansas1d o/p is .xml, for example. My personal preference for the rkh/loq format would be .txt to show that it is ASCII.

“save other” menu is using “append” as default, even though the “append” box is not checked.

Suggest that unless “append” is checked the “filename” changes to the name of the workspace selected.

Good idea.

The “save result” box only gives an obscure error in scripting box if the workspace is no longer there (or I renamed it).

Change History

comment:1 Changed 10 years ago by Steve Williams

(In [6851]) Load and SaveRKH now have default extensions and the save workspaces dialog updates the filename with the workspace's name re #1747

comment:2 Changed 10 years ago by Steve Williams

(In [7306]) Move the MantidWidget class to MantidQT/API, to allow it's functionality to be used by Dialogs, UserSubWindows (interfaces) and widgets re #1747

comment:3 Changed 10 years ago by Martyn Gigg

(In [7310]) Fix the build. Re #1747

comment:4 Changed 10 years ago by Steve Williams

(In [7435]) MantidDialog now replaces MantidQTDialog and the new class has a PythonRunner object re #1747

comment:5 Changed 10 years ago by Steve Williams

(In [7436]) PythonRunner always the runPythonCode() method to be called from the different MantidPlot UI elements that contain one of these objects re #1747

comment:6 Changed 10 years ago by Martyn Gigg

(In [7439]) Fix the MantidPlot build. Re #1747

comment:7 Changed 10 years ago by Steve Williams

(In [7441]) Fix MantidPlot build re #1747

comment:8 Changed 10 years ago by Steve Williams

(In [7443]) Not sure why the MantidPlot build is failing re #1747

comment:9 Changed 10 years ago by Steve Williams

(In [7444]) Fix MantidPlot build attempt, revert previous change re #1747

re #1747

comment:10 Changed 10 years ago by Steve Williams

(In [7451]) Rationalise system test names re #1747

comment:11 Changed 10 years ago by Mathieu Doucet

Note: The new way of running python scripts prevents custom PyQt windows from being displayed within MantidPlot.

comment:12 Changed 10 years ago by Steve Williams

(In [7507]) Improve the SANS GUI error message when trying to save a workspace that doesn't exist re #1747

comment:13 Changed 10 years ago by Steve Williams

(In [7526]) Fix SANS SaveWorkspaces problem, the signal in the base MantidDialog was being obscured by SaveWindow's redeclaration re #1747

comment:14 Changed 10 years ago by Steve Williams

  • Status changed from new to accepted

All the functionality listed in the ticket discription should work now

comment:15 Changed 10 years ago by Steve Williams

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

comment:16 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 26 to Iteration 27

Bulk move of tickets to iteration 27, if your ticket is essential for Iteration 26 then move it back.

comment:17 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 27 to Iteration 26

Sorry I didn't mean to move these ones reverting back to It 26

comment:18 Changed 10 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:19 Changed 10 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Tested files with correct extensions etc.

Tests done on Linux machine and we noted that when loading a .RAW instead of a .raw file the data was not loaded.

comment:20 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 2594

Note: See TracTickets for help on using tickets.