Ticket #5205 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Load algorithm throws exception when load nxspe is present and raw file dialog was the last one

Reported by: Alex Buts Owned by: Roman Tolchenov
Priority: major Milestone: Release 2.1.1
Component: Mantid Keywords: PatchCandidate,Released
Cc: ro Blocked By:
Blocking: Tester: Alex Buts

Description (last modified by Nick Draper) (diff)

The following sequence causes unexpected exception in Mantid:

1) Build nxspe file from a raw file (e.g. MAR11001.raw converted to nxspe, you need MAR11060.raw to do it using Convert To energy interface)

2) Load this MAR11001.raw file 3) remove MAR11001.raw file from Mantid search path. 4) Try to load 11001 file again (if you have MAP as instrument set-up) 5) unexpected exception is thrown suggesting to finish the work.

Change History

comment:1 Changed 8 years ago by Alex Buts

  • Description modified (diff)

comment:2 Changed 8 years ago by Alex Buts

  • Keywords PatchCandidate added

Ann Terry (Sans2D) have risen the same issue when working with Load algorithm. They are typing file names in Load algorithm field and have 2 types of input files (raw and nxs). Claim was that nxs do not work from load but does work from loadNXS. I am almost sure, they hit the issue, revealed by this ticket

comment:3 Changed 8 years ago by Nick Draper

  • Owner set to Roman Tolchenov
  • Status changed from new to assigned

comment:4 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.2 to Release 2.1.1

Moved for inclusion in the 2.1.1 patch

comment:5 Changed 8 years ago by Roman Tolchenov

  • Status changed from assigned to accepted

comment:6 Changed 8 years ago by Roman Tolchenov

Re #5205. LoadDialog can fail if selected file is removed.

Put some protection against that.

Changeset: 837447be7b040020ea7661d1ad7db7e80303e4dd

comment:7 Changed 8 years ago by Roman Tolchenov

Re #5205. Added nxspe extension to Facilities

Changeset: 0dab0da1592cba778c00bb3f1d095f20234af968

comment:8 Changed 8 years ago by Roman Tolchenov

Re #5205. Fixing the tests

Changeset: 8c2dbe94cb1d2282e8c86da1cc3dbc166dbcb50c

comment:9 Changed 8 years ago by Roman Tolchenov

Re #5205. LoadDialog can fail if selected file is removed.

Put some protection against that.

Changeset: 8ce43bfd690adec5b199497a9dc0969e7e7a3965

comment:10 Changed 8 years ago by Roman Tolchenov

Re #5205. Added nxspe extension to Facilities

Changeset: 6856b3582f36a5002715207ce3916aa7973730fc

comment:11 Changed 8 years ago by Roman Tolchenov

Re #5205. Fixing the tests

Changeset: f6a334689b7231b00c013bbc68e209864a1963c9

comment:12 Changed 8 years ago by Roman Tolchenov

Re #5205. LoadDialog can fail if selected file is removed.

Put some protection against that.

Changeset: 837447be7b040020ea7661d1ad7db7e80303e4dd

comment:13 Changed 8 years ago by Roman Tolchenov

Re #5205. Added nxspe extension to Facilities

Changeset: 0dab0da1592cba778c00bb3f1d095f20234af968

comment:14 Changed 8 years ago by Roman Tolchenov

Re #5205. Fixing the tests

Changeset: 8c2dbe94cb1d2282e8c86da1cc3dbc166dbcb50c

comment:15 Changed 8 years ago by Roman Tolchenov

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

comment:16 Changed 8 years ago by Roman Tolchenov

Re #5205. LoadDialog can fail if selected file is removed.

Put some protection against that.

Changeset: 837447be7b040020ea7661d1ad7db7e80303e4dd

comment:17 Changed 8 years ago by Roman Tolchenov

Re #5205. Added nxspe extension to Facilities

Changeset: 0dab0da1592cba778c00bb3f1d095f20234af968

comment:18 Changed 8 years ago by Roman Tolchenov

Re #5205. Fixing the tests

Changeset: 8c2dbe94cb1d2282e8c86da1cc3dbc166dbcb50c

comment:19 Changed 8 years ago by Alex Buts

  • Status changed from verify to verifying
  • Tester set to Alex Buts

comment:20 Changed 8 years ago by Alex Buts

  • Status changed from verifying to reopened
  • Resolution fixed deleted

deleted files works now, but the second option, described in the ticket is still an issue.

comment:21 Changed 8 years ago by Alex Buts

  • Status changed from reopened to assigned
  • Owner changed from Roman Tolchenov to Alex Buts

comment:22 Changed 8 years ago by Alex Buts

  • Status changed from assigned to accepted

comment:23 Changed 8 years ago by Alex Buts

the issue with missing files is dealt with by Roman.

the issue with load algorithm with operations is separate and will be the part of the separate ticket.

comment:24 Changed 8 years ago by Alex Buts

  • Owner changed from Alex Buts to Roman Tolchenov
  • Status changed from accepted to assigned
  • Cc ro added

comment:25 Changed 8 years ago by Nick Draper

  • Description modified (diff)

comment:26 Changed 8 years ago by Nick Draper

  • Status changed from assigned to closed
  • Resolution set to fixed

comment:27 Changed 8 years ago by Peter Parker

Removing comment posted to wrong ticket.

Last edited 8 years ago by Peter Parker (previous) (diff)

comment:28 Changed 8 years ago by Peter Parker

Removing duplicate comment posted to wrong ticket.

Last edited 8 years ago by Peter Parker (previous) (diff)

comment:29 Changed 7 years ago by Nick Draper

  • Keywords PatchCandidate,Released added; PatchCandidate removed

comment:30 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6051

Note: See TracTickets for help on using tickets.