Ticket #5429 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Reflectivity stitching: consistent q binning across polarizations

Reported by: Mathieu Doucet Owned by: Mathieu Doucet
Priority: major Milestone: Release 2.3
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description

The data stitching for reflectivity trims the zeros on both sides of each curve segment to be stitched together. When a particular q-value has more zeros then other polarization states, the resulting binning for that stitched data may be different. Make sure that points are trimmed the same way across all polarization states.

Change History

comment:1 Changed 8 years ago by Mathieu Doucet

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Mathieu Doucet

Re #5429 fix problem with varying binning

Changeset: 2b1d79caf0029ef6607bd5e34fc4deea171ca20f

comment:3 Changed 8 years ago by Mathieu Doucet

Re #5429 Restrict TOF range when plotting raw data in 2D

Changeset: e8f1f3a9db87f7ca9219aafec05a469a91e604d5

comment:4 Changed 8 years ago by Mathieu Doucet

Re #5429 fix problem with varying binning

Changeset: 2b1d79caf0029ef6607bd5e34fc4deea171ca20f

comment:5 Changed 8 years ago by Mathieu Doucet

Re #5429 Restrict TOF range when plotting raw data in 2D

Changeset: e8f1f3a9db87f7ca9219aafec05a469a91e604d5

comment:6 Changed 8 years ago by Mathieu Doucet

Re #5429 fix problem with varying binning

Changeset: 2b1d79caf0029ef6607bd5e34fc4deea171ca20f

comment:7 Changed 8 years ago by Mathieu Doucet

Re #5429 Restrict TOF range when plotting raw data in 2D

Changeset: e8f1f3a9db87f7ca9219aafec05a469a91e604d5

comment:8 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.2 to Release 2.3

Moved at the end of release 2.2

comment:9 Changed 8 years ago by Mathieu Doucet

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

This now works.

comment:10 Changed 8 years ago by Wenduo Zhou

Last edited 8 years ago by Russell Taylor (previous) (diff)

comment:11 Changed 8 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:12 Changed 8 years ago by Russell Taylor

  • Status changed from verifying to closed

This has been in production for a long time, so has effectively been tested by the users. Jean says it's good.

comment:13 Changed 8 years ago by Wenduo Zhou

Fix unit test. Refs #5429.

Changeset: 83767ed8e9717641ea71c24ee83353ad678a71de

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6275

Note: See TracTickets for help on using tickets.