Ticket #10513 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Calibration preview plot show pre and post smooth

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: Martyn Gigg

Description (last modified by Dan Nixon) (diff)

The plot should show the output pre and post WienerSmooth.

This will most likely mean moving the smooth back out of the resolution algorithm and doing it as part of the algorithm batch in the UI.

Change History

comment:1 Changed 6 years ago by Dan Nixon

  • Status changed from new to assigned
  • Component changed from Direct Inelastic to Indirect Inelastic

comment:2 Changed 6 years ago by Dan Nixon

  • Milestone changed from Backlog to Release 3.4

comment:3 Changed 6 years ago by Dan Nixon

  • Description modified (diff)

comment:4 Changed 6 years ago by Dan Nixon

  • Status changed from assigned to inprogress

Fix UI issues in calibration tab

Refs #10513

Changeset: 6ae76a4695198ea69d5f24fb53d028e17d5af271

comment:5 Changed 6 years ago by Dan Nixon

  • Description modified (diff)

comment:6 Changed 6 years ago by Dan Nixon

Support plotting both pre and post scale on res

Refs #10513

Changeset: 995ce4a0b2b40dbbbf13d977f60447ef636e0546

comment:7 Changed 6 years ago by Dan Nixon

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

This is being verified as ​pull request #171.

comment:8 Changed 6 years ago by Dan Nixon

This also does not save the correct workspace when smooth is enabled.

comment:9 Changed 6 years ago by Dan Nixon

Merge branch 'master' into feature/10513_calibration_show_pre_post_smooth

Conflicts:

Code/Mantid/MantidQt/CustomInterfaces/src/Indirect/IndirectCalibration.cpp

Refs #10513

Changeset: 1cfcd0b4247f7b9c30e02d65febbdf3d7d722c90

comment:10 Changed 6 years ago by Dan Nixon

Add save to smooth calibration

Refs #10513

Changeset: 72c396dede5b88f121875fc4bd5c3e7069638320

comment:11 Changed 6 years ago by Dan Nixon

Jenkins, retest this please.

comment:12 Changed 6 years ago by Martyn Gigg

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

comment:13 Changed 6 years ago by Martyn Gigg

I've played around with the tab and as far as I can tell it is doing the correct thing.

comment:14 Changed 6 years ago by Martyn Gigg

  • Status changed from verifying to closed

Merge pull request #171 from mantidproject/feature/10513_calibration_show_pre_post_smooth

IDR calibration show pre and post smooth

Full changeset: ​fbe8a6f8dd16bd42ed10ce466f03bc1116fac3ed

comment:15 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​11355

Note: See TracTickets for help on using tickets.