Ticket #7953 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Make MultipleScatteringCylinderAbsorption event aware

Reported by: Peter Peterson Owned by: Peter Peterson
Priority: major Milestone: Release 3.0
Component: Framework Keywords:
Cc: Blocked By:
Blocking: #7987 Tester: Michael Reuter

Description (last modified by Peter Peterson) (diff)

This is the first step in a real push to get absorption corrected for in mantid.

Attachments

pg3van15138compare.py (2.2 KB) - added by Peter Peterson 7 years ago.

Change History

comment:1 Changed 7 years ago by Peter Peterson

  • Status changed from new to inprogress

comment:2 Changed 7 years ago by Peter Peterson

  • Description modified (diff)

comment:3 Changed 7 years ago by Peter Peterson

  • Blocking 7987 added

comment:4 Changed 7 years ago by Peter Peterson

  • Description modified (diff)

Moved some of the work to a new ticket to make this easier to verify.

comment:5 Changed 7 years ago by Peter Peterson

Re #7953. Using SetSampleMaterial to clarify things.

Changeset: aa4a9c58fca26806ce079f3935cb756cbc454178

comment:6 Changed 7 years ago by Peter Peterson

Re #7953. Adding more logging and fixing but in absorptionXS get.

Changeset: 1035c29ba18d38b04e0e728d9c2fa9d0a2c0ca33

comment:7 Changed 7 years ago by Peter Peterson

Re #7953. Quieting compiler warning.

Changeset: 8058fd6973337ad43584325d8ea27c25968edb1a

comment:8 Changed 7 years ago by Peter Peterson

Re #7953. Changed variable name to reflect actual value.

Changeset: f4b2105f63683d21655589ba0a609899e7890c27

comment:9 Changed 7 years ago by Peter Peterson

Re #7953. Made MultScatt... event aware.

Changeset: 555f0b79523a4a1373152d7b387f00ae76960108

comment:10 Changed 7 years ago by Peter Peterson

Re #7953. Moved implementation from header to source.

Changeset: 71482bf3e847d554115e38b513d46437b4333a07

comment:11 Changed 7 years ago by Peter Peterson

Re #7953. Added in progress bar reporting.

Changeset: 02fc1bfe14e096ac45caa26a5e4877db8e7777e6

comment:12 Changed 7 years ago by Peter Peterson

This is on feature/7953_MultipleScatteringCylinderAbsorption.

To test: Run the attached script and compare the various reductions.

Changed 7 years ago by Peter Peterson

comment:13 Changed 7 years ago by Peter Peterson

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

comment:14 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:15 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to reopened
  • Resolution fixed deleted

This needs a unit test for the events.

comment:16 Changed 7 years ago by Peter Peterson

  • Status changed from reopened to inprogress

Re #7953. Adding unit test for the event case of MultScattCylAbs...

Really easy case to pass. Mostly just confirms that you get an event workspace back. Since the math is the same between both event and histo rely on the histogram to keep the calculation correct.

Changeset: 3e7645ee40c9db421e91050e30694bd7ca0d22fc

comment:17 Changed 7 years ago by Peter Peterson

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

Same testing regiment as mentioned in comment:12.

comment:18 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying

comment:19 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/7953_MultipleScatteringCylinderAbsorption'

comment:20 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8798

Note: See TracTickets for help on using tickets.