Ticket #4878 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

[IDA] ConvFit - Unexpected Exception When Plotting

Reported by: Peter Parker Owned by: Peter Parker
Priority: major Milestone: Release 2.1
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Stuart Campbell

Description (last modified by Peter Parker) (diff)

To reproduce, use the attached files to set up the ConvFit tab of the IDA interface as it appears in the screenshot. The value of "Use" in the property table must be set to true, and "Plot Guess" must also be checked.

Clicking anywhere in the plotted graph should throw the error.

Attachments

osi94909_graphite002_red.nxs​ (1018.4 KB) - added by Peter Parker 9 years ago.
osi94910_graphite002_res.nxs​ (195.7 KB) - added by Peter Parker 9 years ago.
Screenshot.png​ (495.2 KB) - added by Peter Parker 9 years ago.

Change History

Changed 9 years ago by Peter Parker

Changed 9 years ago by Peter Parker

Changed 9 years ago by Peter Parker

comment:1 Changed 9 years ago by Peter Parker

  • Description modified (diff)

comment:2 Changed 8 years ago by Peter Parker

Refs #4878 - Fix unhandled exception.

Rejigged functions to better handled user input, and to fix case where we were throwing an unhandled exception.

Added comments to make code clearer.

Changeset: f6abbfe842828f08b3e45bb7872cc3b8c4cdad66

comment:3 Changed 8 years ago by Peter Parker

Refs #4878 - Accommodate recent changes to Fit.

The Function property of the Fit algorithm must now be set before the InputWorkspace property.

Parameter names are now taken straight from the function instead of the algorithm.

Changeset: c67f732a3dfd344d8c1becb81bdd5b6cd3f5737e

comment:4 Changed 8 years ago by Peter Parker

Refs #4878 - Fix unhandled exception.

Rejigged functions to better handled user input, and to fix case where we were throwing an unhandled exception.

Added comments to make code clearer.

Changeset: f6abbfe842828f08b3e45bb7872cc3b8c4cdad66

comment:5 Changed 8 years ago by Peter Parker

Refs #4878 - Accommodate recent changes to Fit.

The Function property of the Fit algorithm must now be set before the InputWorkspace property.

Parameter names are now taken straight from the function instead of the algorithm.

Changeset: c67f732a3dfd344d8c1becb81bdd5b6cd3f5737e

comment:6 Changed 8 years ago by Peter Parker

  • Status changed from new to accepted

comment:7 Changed 8 years ago by Peter Parker

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

Closing.

To test, make sure that the exception in the ticket description does not occur.

comment:8 Changed 8 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:9 Changed 8 years ago by Stuart Campbell

I still get an exception, but with the error...

Unexpected exception: ParamFunction DeltaFunction does not have parameter (f0.Height).

Last edited 8 years ago by Stuart Campbell (previous) (diff)

comment:10 Changed 8 years ago by Stuart Campbell

  • Status changed from verifying to reopened
  • Resolution fixed deleted

comment:11 Changed 8 years ago by Peter Parker

Refs #4878 - Actually fixing error this time ...

Changeset: 61b674e6ce9e2e9fe29e8788b0d130b65f193a29

comment:12 Changed 8 years ago by Peter Parker

  • Status changed from reopened to accepted

comment:13 Changed 8 years ago by Peter Parker

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

comment:14 Changed 8 years ago by Stuart Campbell

  • Status changed from verify to verifying

comment:15 Changed 8 years ago by Stuart Campbell

  • Status changed from verifying to closed

Looks good.

comment:16 Changed 8 years ago by Peter Parker

Refs #4878 - Actually fixing error this time ...

Changeset: 61b674e6ce9e2e9fe29e8788b0d130b65f193a29

comment:17 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5725

Note: See TracTickets for help on using tickets.