Ticket #6445 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Error message from Load is a little complex

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 2.6
Component: Framework Keywords: Maintenance
Cc: Blocked By:
Blocking: Tester: Karl Palmen

Description

Try running 

ws = Load('MAR1101.RA')

you get the error 

ValueError: Invalid value for property Filename (list of str lists) "MAR1101.RA": Unable to parse runs: "Unable to find file matching the string "MAR01101.RA": "File "MAR01101.RA" not found".". 
  at line 1 in ''
  caused by line 88 in '/home/dmn58364/Builds/mantid/debug/bin/mantid/simpleapi.py'

It seems an overly complex message to simply say that the file can't be found.

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.5 to Release 2.6

Moved to r2.6 at the end of r2.5

comment:2 Changed 7 years ago by Nick Draper

  • Owner set to Martyn Gigg
  • Status changed from new to assigned
  • Keywords Maintenance added

comment:3 Changed 7 years ago by Martyn Gigg

  • Status changed from assigned to accepted

comment:4 Changed 7 years ago by Martyn Gigg

Simplfy error message from Load when files are not found. Refs #6445

Changeset: 74ec49455ef0b47e7aa96208623244455e652963

comment:5 Changed 7 years ago by Martyn Gigg

Improve error msg when a file from a range is not found. Refs #6445

Changeset: 414e7fe7e4a53f975f4c3f601cccef08a4ed639d

comment:6 Changed 7 years ago by Martyn Gigg

Merge branch 'feature/6445_error_message_from_Load' into develop

comment:7 Changed 7 years ago by Martyn Gigg

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

Branch: feature/6445_error_message_from_Load

Tester:

  • Check that Load("MAR1101.RA") gives an error message that is more to the point, i.e it can't find the file
  • Set your default instrument to MUSR (at ISIS)
  • Check that Load("15188-15190") gives an error and that the error relates to not being able to find MUSR1588 and not a lexical cast error (as it was giving)
  • Also check success cases i.e. Load("15189-15191"), Load("MAR11001.raw") etc

comment:8 Changed 7 years ago by Martyn Gigg

Merge branch 'feature/6445_error_message_from_Load' into develop into 6856_ConvertToDiffractionMDWS_v2

comment:9 Changed 7 years ago by Martyn Gigg

Merge branch 'feature/6445_error_message_from_Load' into develop into 6856_ConvertToDiffractionMDWS_v2

comment:10 Changed 7 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:11 Changed 7 years ago by Karl Palmen

  • Status changed from verifying to closed

comment:12 Changed 7 years ago by Karl Palmen

Merge remote-tracking branch 'origin/feature/6445_error_message_from_Load' into master

comment:13 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7291

Note: See TracTickets for help on using tickets.