Ticket #6062 (closed: worksforme)

Opened 8 years ago

Last modified 5 years ago

Multiple files loading for SNS

Reported by: Andrei Savici Owned by: Roman Tolchenov
Priority: major Milestone: Release 3.1
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description (last modified by Roman Tolchenov) (diff)

I am trying to do something like 

w=Load('ARCS30000-30002')

and I get 

RuntimeError: Instrumentsearch object ARCS30000-
   at line 1 in '__main__'
   caused by line 85 in '/home/3y9/Mantid/Qt-build/bin/mantid/simpleapi.py'

Change History

comment:1 Changed 8 years ago by Roman Tolchenov

  • Status changed from new to accepted
  • Owner changed from Russell Taylor to Roman Tolchenov
  • Milestone changed from Release 2.4 to Release 2.3

comment:2 Changed 8 years ago by Roman Tolchenov

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

#6082 should also fix this ticket.

comment:3 Changed 8 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:4 Changed 8 years ago by Martyn Gigg

  • Status changed from verifying to reopened
  • Resolution fixed deleted
  • Milestone changed from Release 2.3 to Release 2.4

This still doesn't work. I didn't have ARCS files so instead I used the CNCS_7860_event.nxs from the systemtests Data directory and made 3 copies called CNCS_7861_event.nxs ,CNCS_7862_event.nxs ,CNCS_7863_event.nxs and tried 

ws = Load('CNCS7860-7863')

and it produced 

RuntimeError: Instrument search object CNCS7860-
  at line 1 in ''
  caused by line 85 in '/home/dmn58364/Builds/mantid/release/bin/mantid/simpleapi.py'

This is not considered high enough priority to be fixed for 2.3. Moved to 2.4

Last edited 8 years ago by Martyn Gigg (previous) (diff)

comment:5 Changed 8 years ago by Russell Taylor

Another thing, that seems to be related to this because of the error message, is that if your default instrument is an SNS one and you give an incorrect run number to python Load, e.g. ws = Load('7866') then the error message that comes back does not tell you that it can't find the file: 

ValueError: Invalid value for property Filename (list of str lists) "7862": Unable to parse runs: "Instrument search object CNCS_". 
  at line 1 in '__main__'
  caused by line 85 in '/home/tr9/Mantid/Code/debug/bin/mantid/simpleapi.py'

For an ISIS instrument it more helpfully says: "Unable to find file matching the string "HRP12345", even after appending suggested file extensions."

comment:6 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.4 to Release 2.5

Moved at the code freeze for release 2.4

comment:7 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:8 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:9 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.6 to Backlog

Moved to the Backlog after the code freeze for R2.6

comment:10 Changed 7 years ago by Nick Draper

  • Milestone changed from Backlog to Release 3.0

moved to Release 3.0 as these all seem to be active

comment:11 Changed 7 years ago by Roman Tolchenov

  • Description modified (diff)
  • Milestone changed from Release 3.0 to Release 3.1

comment:12 Changed 7 years ago by Roman Tolchenov

  • Status changed from reopened to verify
  • Resolution set to worksforme

It seems the issue has been fixed. Tried the example from comment 4 and it worked although there were two output workspaces which is odd.

The error message for the example in comment 5 is the same for an SNS and an ISIS instruments and reads "Unable to find file ..."

comment:13 Changed 7 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester changed from Martyn Gigg to Russell Taylor

comment:14 Changed 7 years ago by Russell Taylor

Confirmed that there's no longer an error.

Does anyone know if there's a ticket already for the fact that multiple workspaces are created? (You actually end up with 3: the one you asked for, an identical one with a name like ARCS_30000_event_ARCS_30001_event_ARCS_30002_event and one called __@loadsum_temp@ which is presumably used to hold an individual file before summation and should be cleaned up.)

Last edited 7 years ago by Russell Taylor (previous) (diff)

comment:15 Changed 7 years ago by Russell Taylor

  • Status changed from verifying to closed

#8556 has been created regarding the multiple output workspaces issue.

comment:16 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6908

Note: See TracTickets for help on using tickets.