Ticket #2820 (closed: duplicate)

Opened 9 years ago

Last modified 5 years ago

Bug in getPhi routine for Grouped detectors

Reported by: Laurent Chapon Owned by: Russell Taylor
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description

The getPhi routine does not quite work in special cases for Grouped Detectors. For example if you have 4 pixels very close to the equatorial plane at phi=180 deg, with say 2 positive and 2 negative values of phi (180,-180), then the sum is zero whether it should be close to 180 deg.

Change History

comment:1 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 28 to Iteration 29

Bulk move of tickets at the end of iteration 28

comment:2 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 29 to Iteration 30

"New" tickets moved at the code freeze of iteration 29

comment:3 Changed 9 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner changed from Laurent Chapon to Russell Taylor

First just test out if this is true and testable

comment:4 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 30 to Iteration 31

Bulk move of tickets to iteration 31 at the iteration 30 code freeze

comment:5 Changed 9 years ago by Russell Taylor

  • type changed from enhancement to defect

comment:6 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 32 to Iteration 33

Moved to iteration 33 at iteration 32 code freeze

comment:7 Changed 9 years ago by Russell Taylor

  • Status changed from assigned to accepted
  • Milestone changed from Iteration 33 to Iteration 32

comment:8 Changed 9 years ago by Russell Taylor

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

This has been fixed in #3923.

comment:9 Changed 9 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:10 Changed 9 years ago by Nick Draper

  • Status changed from verifying to closed

duplicate agreed

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3667

Note: See TracTickets for help on using tickets.