Ticket #8793 (closed: fixed)
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: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
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