Ticket #3052 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Change the NearestNeighbours code to work on spectra rather than pixels

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: critical Milestone: Iteration 29
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Roman Tolchenov

Description

An instrument can have detectors grouped at the hardware level, i.e WISH and MERLIN, so the nearest neighbours code needs to change to find nearest spectra rather then nearest detectors.

Change History

comment:1 Changed 9 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Martyn Gigg

(In [11920]) Added an interface for the spectra-detector map and updated it with iterator capabilities. Also added a width member to BoundingBox. Refs #3052

comment:3 Changed 9 years ago by Martyn Gigg

(In [11925]) Consolidate some code (well it will be when I commit the bit that uses it later). Re #3052

comment:4 Changed 9 years ago by Martyn Gigg

(In [11949]) Re #3052. Nearly final touches to changes for nearest neighbours grouping spectra not detectors. It still returns det_ids which isn't ideal but the core functionality is doing what is should do so this is essentially a checkpoint.

comment:5 Changed 9 years ago by Martyn Gigg

(In [11965]) The nearest neighbours method now on the detector class now returns a mapping of spectrum number to distance rather than detector ID. Re #3052

comment:6 Changed 9 years ago by Martyn Gigg

(In [11966]) Fix PythonAPI build Re #3052

comment:7 Changed 9 years ago by Martyn Gigg

(In [11974]) Change the boost graph in the nearest neighbours class to store the distances in real space not our scaled version. Re #3052

comment:8 Changed 9 years ago by Martyn Gigg

(In [11976]) Fix for crash in release mode where an array went out of bounds. Re #3052

comment:9 Changed 9 years ago by Martyn Gigg

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

comment:10 Changed 9 years ago by Roman Tolchenov

  • Status changed from verify to verifying
  • Tester set to Roman Tolchenov

comment:11 Changed 9 years ago by Roman Tolchenov

  • Status changed from verifying to closed

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3899

Note: See TracTickets for help on using tickets.