Ticket #8793 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

ConvFit plots incorrect spectra in each plot.

Reported by: Samuel Jackson Owned by: Samuel Jackson
Priority: minor Milestone: Release 3.1
Component: Indirect Inelastic Keywords:
Cc: Blocked By:
Blocking: Tester: Gesner Passos

Description

ConvFit is plotting Amplitude and FWHM together for each peak, it should be plotting Amplitude for both and width for both.

Change History

comment:1 Changed 7 years ago by Samuel Jackson

  • Status changed from new to inprogress

Refs #8793 Move multi spectra plotting code to IndirectCommon.

We already have code that handles this fairly well in Bayes. We'll just share it between the two.

Changeset: 541d474239d1cbb85049d73adfc0ac52d788a250

comment:2 Changed 7 years ago by Samuel Jackson

Refs #8793 Remove duplicated code in IndirectBayes.

This has now been moved to IndirectCommon.

Changeset: 674843638e7cdcf708b9d7e1e564f6fd54fb4a3b

comment:3 Changed 7 years ago by Samuel Jackson

Refs #8793 Change ConvFit to plot either FWHM or Amplitude

Changeset: 9f8369dc5d24b00e563380e7cafce67165361831

comment:4 Changed 7 years ago by Samuel Jackson

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

To Tester

Run a sequential ConvFit with two Lorentzians and try all of the plotting options. You should get plots showing all the Amplitudes and all the FWHMs depending on plotting choice.

comment:5 Changed 7 years ago by Gesner Passos

  • Status changed from verify to verifying
  • Tester set to Gesner Passos

comment:6 Changed 7 years ago by Gesner Passos

Implement the requirement, and improve code sharing. ;)

comment:7 Changed 7 years ago by Gesner Passos

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/8793_convfit_plotting'

Full changeset: 76998af1ced5c07f4400309da0ab021861eefec3

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9637

Note: See TracTickets for help on using tickets.