Ticket #11700 (closed: fixed)
Add monitor 2 to VESUVIO IPF
Reported by: | Dan Nixon | Owned by: | Dan Nixon |
---|---|---|---|
Priority: | major | Milestone: | Release 3.5 |
Component: | Indirect Inelastic | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Federico Montesino Pouzols |
Description (last modified by Dan Nixon) (diff)
This will allow use of VESUVIO in the IndirectTransmissionMonitor algorithm.
Change History
comment:4 Changed 5 years ago by Dan Nixon
Use workflow parameters to determine monitors
Refs #11700
Changeset: 5c6d3ebb6c294a59fcf83bcfa0b24a5f2c041461
comment:5 Changed 5 years ago by Dan Nixon
- Status changed from inprogress to verify
- Resolution set to fixed
This is being verified as pull request #718.
comment:6 Changed 5 years ago by Federico Montesino Pouzols
- Status changed from verify to verifying
- Tester set to Federico Montesino Pouzols
comment:7 Changed 5 years ago by Federico Montesino Pouzols
It seems to work well and results look sensible as far as I can tell. I only noticed that when you use the Plot option you'll get two warnings. In principle the plots look fine, but just in case, it says: ` You are overlaying plots from data having differing units! You are overlaying distribution and non-distribution data! `
comment:8 Changed 5 years ago by Federico Montesino Pouzols
- Status changed from verifying to closed
Merge pull request #718 from mantidproject/11700_VESUVIO_monitor2_IPF
Support VESUVIO on IndirectTransmissionMonitor
Full changeset: 566047084a3f25a497696bc7d3532d76f777460d