Ticket #11661 (closed: fixed)

Opened 5 years ago

Last modified 5 years ago

IDA CalcCorr beam heigh takes beam width value

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: Federico Montesino Pouzols

Description (last modified by Dan Nixon) (diff)

There is a parameter Workflow.beam-height for this.

Change History

comment:1 Changed 5 years ago by Dan Nixon

  • Description modified (diff)

comment:2 Changed 5 years ago by Dan Nixon

  • Cc spencer.howells@… added
  • Status changed from new to assigned

comment:3 Changed 5 years ago by Dan Nixon

  • Status changed from assigned to inprogress

Use correct beam height value in Calc Corr

Refs #11661

Changeset: 90c560b1752b254892a55bdf805b3afe231283c1

comment:4 Changed 5 years ago by Dan Nixon

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

This is being verified as ​pull request #684.

comment:5 Changed 5 years ago by Dan Nixon

  • Milestone changed from Release 3.5 to Release 3.4

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

Worked well for me. I used an OSIRIS _red.nxs file and while with master I get a wrong height value (2.2), with this branch I get the proper width/height (2.2/4.4) values from the current IPF, both for Cylinder and Annulus shapes.

comment:8 Changed 5 years ago by Federico Montesino Pouzols

  • Status changed from verifying to closed

Merge pull request #684 from mantidproject/11661_calccorr_use_correct_beam_height

Use correct beam height value in Calc Corr

Full changeset: ​1993eccfc7a848961b41e3f0de532e27c5c89e5b

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​12499

Note: See TracTickets for help on using tickets.