Ticket #11253 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

CalcCorr issues

Reported by: Dan Nixon Owned by: Dan Nixon
Priority: major Milestone: Release 3.4
Component: Indirect Inelastic Keywords:
Cc: spencer.howells@… Blocked By:
Blocking: Tester: Federico Montesino Pouzols

Description

Couple of issues reported by Spencer: 

On opening tab, the geometry is Flat but it shows the Cyl  radii,etc. Changing to Cyl & back to Flat gives the correct thickness,etc.
For cross-sections, although default is Formula it still insists on wanting to input cross-sections (red asterix).

Change History

comment:1 Changed 6 years ago by Dan Nixon

  • Cc spencer.howells@… added
  • Status changed from new to assigned

comment:2 Changed 6 years ago by Dan Nixon

  • Component changed from Framework to Indirect Inelastic

comment:3 Changed 6 years ago by Dan Nixon

  • Status changed from assigned to inprogress

Fix validation for cross section source

Refs #11253

Changeset: a8b349f639b727161f86ba5c5b0d30c8861d265d

comment:4 Changed 6 years ago by Dan Nixon

Set correct default UI options

Refs #11253

Changeset: f9e522db1b650aa89e100aa981cc45a35022ac3c

comment:5 Changed 6 years ago by Dan Nixon

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

This is being verified as ​pull request #395.

comment:6 Changed 6 years ago by Dan Nixon

Jenkins, retest this please.

comment:7 Changed 6 years ago by Federico Montesino Pouzols

  • Status changed from verify to verifying
  • Tester set to Federico Montesino Pouzols

comment:8 Changed 6 years ago by Federico Montesino Pouzols

It seems to behave well. I only noticed the very minor/picky issue that when you choose to 'plot output' the titles of the plots are not very informative (they're named Graph1, Graph2, etc.).

comment:9 Changed 6 years ago by Federico Montesino Pouzols

  • Status changed from verifying to closed

Merge pull request #395 from mantidproject/11253_calccorr_ui_issues

IDA CalcCor UI issues

Full changeset: ​48d0bdb3edefe13aba49bf7c3e144f4d05243b3d

comment:10 Changed 6 years ago by Dan Nixon

@FedeMPouzols That should have been addressed in a new ticket that replaces the Python code behind this tab. It will be plotting a workspace group so will default to the workspace title IIRC.

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​12092

Note: See TracTickets for help on using tickets.