Ticket #4153 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Allow Load to work with unknown properties.

Reported by: Stuart Campbell Owned by: Stuart Campbell
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By: #4151
Blocking: Tester: Nick Draper

Description (last modified by Stuart Campbell) (diff)

At the moment if you run Load with some properties that are not recognised by the underlying algorithm then you get a runtime error. This should be changed to be a warning message so we can make use of Load more generally.

Change History

comment:1 Changed 9 years ago by Stuart Campbell

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Stuart Campbell

Added additional methods to send different log level messages. refs #4151

Changeset: 7f5862c3394e572d5ba176e86833fdcb1eb90e5d

comment:3 Changed 9 years ago by Stuart Campbell

  • Description modified (diff)

comment:4 Changed 9 years ago by Stuart Campbell

  • Summary changed from Allow Load to work with unknown parameters. to Allow Load to work with unknown properties.

comment:5 Changed 9 years ago by Stuart Campbell

Added more detail to output warning message. refs #4153

Now include the name of the property that we are trying to set.

Changeset: e7438f3f23cbd5135e439cbc083364b888bf1313

comment:6 Changed 9 years ago by Stuart Campbell

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

We may want a better long term solution, but this works for now.

comment:7 Changed 9 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

tested with the following line 

a=Load("GEM1101",bob="ill")

verified that the algorithm continues, with the following message at information level 

You've passed a property (bob) to Load() that doesn't apply to this file type.

comment:8 Changed 9 years ago by Nick Draper

  • Status changed from verifying to closed

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5000

Note: See TracTickets for help on using tickets.