Ticket #6499 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Better File selection for UpdateInstrumentFromFile

Reported by: Karl Palmen Owned by: Karl Palmen
Priority: major Milestone: Release 2.5
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Anders Markvardsen

Description

Currently the file browser to select the file for UpdateInstrumentFromFile only has *.raw, *.s and all in its filetype menu. *.nxs could be included along with other changes to update it.

Also if a Nexus file that is not ISIS Nexus (e.g. Processed Nexus) is found the error message is that *.nxs is an invalid file extension. A more informative error message would be useful.

Change History

comment:1 Changed 8 years ago by Karl Palmen

  • Status changed from new to assigned
  • Owner set to Karl Palmen

comment:2 Changed 8 years ago by Karl Palmen

  • Status changed from assigned to accepted

comment:3 Changed 8 years ago by Karl Palmen

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

I've now added *.nxs to the file types. If the Nexus file is not ISIS nexus. It will say that the file (rather than file type) is not valid. Hopefully the user would go to the documentation to find out why.

comment:4 Changed 8 years ago by Karl Palmen

To test,

load a workspace.

Execute UpdateInstrumentFromFile in Mantidplot.

Browse for a file.

Verify that *.nxs as available as a file type.

Choose a Nexus file for the same instrument as the workspace loaded, but is not ISIS nexus. This can be a processed nexus file that you have saved from the loaded worskpace.

Check error message refers to invalid file rather than invalid file type.

comment:5 Changed 8 years ago by Anders Markvardsen

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

comment:6 Changed 8 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Tested this on EMU data and MAPS and veriry *.nxs is available and error message output is 'not a valid input file' rather than type.

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7345

Note: See TracTickets for help on using tickets.