Ticket #5346 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Verify NOMAD low angle bank geometry

Reported by: Peter Peterson Owned by: Peter Peterson
Priority: critical Milestone: Release 2.1.1
Component: Mantid Keywords: PatchCandidate,Released
Cc: Blocked By:
Blocking: Tester: Vickie Lynch

Description

Comparing Joerg's IDL code with mantid it appears as though the geometry for the low angle detectors is incorrect. Verify that it is correct or fix it.

Change History

comment:1 Changed 8 years ago by Peter Peterson

The idl version of the code is at /SNS/NOM/shared/idl

comment:2 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.2 to Release 2.1.1

Moved for inclusion in the 2.1.1 patch

comment:3 Changed 8 years ago by Peter Peterson

Refs #5346. New NOMAD geometries.

1900-00-00 to 2012-06-30 is the original geometry. 2012-07-01 to 2012-07-31 is the geometry with various fixes. without extension is the fixed geometry with predicted correct detector ids.

Changeset: 7471ce9b5657ee755fe22462d2a3c32773e4800a

comment:4 Changed 8 years ago by Peter Peterson

  • Status changed from new to accepted

comment:5 Changed 8 years ago by Peter Peterson

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

comment:6 Changed 8 years ago by Peter Peterson

Refs #5346. Renamed old geometry to point out its new temporal nature.

Changeset: dd1baaa3589aff3b296e4ae9ce628ebbe8806e42

comment:7 Changed 8 years ago by Peter Peterson

Refs #5346. Changed effective dates for NOMAD.

Changeset: cf9011026a2816382db75654d26f017ccc0dc277

comment:8 Changed 8 years ago by Peter Peterson

Refs #5346. Copying to new NOMAD instrument definition.

Changeset: c1f3790ac7542b5b2692bc73b92107722460d33f

comment:9 Changed 8 years ago by Peter Peterson

Refs #5346. Didn't get the right version in the master.

Changeset: ab6f4d7e03975b0b7814cd693a7a9ad5ae0485a3

comment:10 Changed 8 years ago by Vickie Lynch

  • Status changed from verify to verifying
  • Tester set to Vickie Lynch

comment:11 Changed 8 years ago by Peter Peterson

Refs #5346. Fixing descrepancy that Vickie found in testing.

Changeset: 353a2678c2ea6750dab2d570b39e9d01626e86e2

comment:12 Changed 8 years ago by Vickie Lynch

  • Status changed from verifying to closed

LoadEventNexus with NOM_3887 and then LoadInstrument with both new IDFs and they look good.

comment:13 Changed 8 years ago by Peter Peterson

Refs #5346. Renamed old geometry to point out its new temporal nature.

Changeset: dd1baaa3589aff3b296e4ae9ce628ebbe8806e42

comment:14 Changed 8 years ago by Peter Peterson

Refs #5346. Changed effective dates for NOMAD.

Changeset: cf9011026a2816382db75654d26f017ccc0dc277

comment:15 Changed 8 years ago by Peter Peterson

Refs #5346. Copying to new NOMAD instrument definition.

Changeset: c1f3790ac7542b5b2692bc73b92107722460d33f

comment:16 Changed 8 years ago by Peter Peterson

Refs #5346. Didn't get the right version in the master.

Changeset: ab6f4d7e03975b0b7814cd693a7a9ad5ae0485a3

comment:17 Changed 8 years ago by Vickie Lynch

Refs #5346 Same NOMAD geometry as Release 2.1.1

Changeset: 3c3d297166d5e81d60428e07bc7bb19108a472b1

comment:18 Changed 8 years ago by Vickie Lynch

Refs #5346 Same NOMAD geometry as Release 2.1.1

Changeset: 3c3d297166d5e81d60428e07bc7bb19108a472b1

comment:19 Changed 8 years ago by Peter Peterson

Refs #5346. Renamed old geometry to point out its new temporal nature.

Changeset: dd1baaa3589aff3b296e4ae9ce628ebbe8806e42

comment:20 Changed 8 years ago by Peter Peterson

Refs #5346. Changed effective dates for NOMAD.

Changeset: cf9011026a2816382db75654d26f017ccc0dc277

comment:21 Changed 8 years ago by Peter Peterson

Refs #5346. Copying to new NOMAD instrument definition.

Changeset: c1f3790ac7542b5b2692bc73b92107722460d33f

comment:22 Changed 8 years ago by Peter Peterson

Refs #5346. Didn't get the right version in the master.

Changeset: ab6f4d7e03975b0b7814cd693a7a9ad5ae0485a3

comment:23 Changed 8 years ago by Vickie Lynch

Refs #5346 Same NOMAD geometry as Release 2.1.1

Changeset: 3c3d297166d5e81d60428e07bc7bb19108a472b1

comment:24 Changed 7 years ago by Nick Draper

  • Keywords PatchCandidate,Released added; PatchCandidate removed

comment:25 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6192

Note: See TracTickets for help on using tickets.