Ticket #9441 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Improve memory usage in CalibrateRectangularDetector

Reported by: Peter Peterson Owned by: Peter Peterson
Priority: trivial Milestone: Release 3.2
Component: Diffraction Keywords:
Cc: Blocked By:
Blocking: Tester: Vickie Lynch

Description

The algorithm never even calls CompressEvents. With very little work it can use a lot less memory.

Change History

comment:1 Changed 6 years ago by Peter Peterson

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Pete Peterson

  • Status changed from assigned to inprogress

Re #9441. Adding some CompressEvents calls.

Changeset: bba4d718839d78d92b07fface27c51c4fcee62ef

comment:3 Changed 6 years ago by Pete Peterson

Re #9441. Removing support for histogram files.

Changeset: 2d3ea31503304acd939887d88153f3cc0f2d279e

comment:4 Changed 6 years ago by Pete Peterson

Re #9441. Removing option to filter by log value.

Changeset: 70d6dc3898aa86d48995d9197cc27072a0528d02

comment:5 Changed 6 years ago by Pete Peterson

Re #9441. Removing extra compressions

Changeset: e9056b42f5ec37ed8ac8678ebe19abc391895503

comment:6 Changed 6 years ago by Peter Peterson

This is on feature/9441_memory_in_calib.

To test: Do a calibration before and after merging. They should be identical as this doesn't have any changes other than removing dead code and calling CompressEvents.

comment:7 Changed 6 years ago by Peter Peterson

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

comment:8 Changed 6 years ago by Vickie Lynch

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

comment:9 Changed 6 years ago by Vickie Lynch

  • Status changed from verifying to closed

Merge remote branch 'origin/feature/9441_memory_in_calib'

Full changeset: 26609dbae7130f79e816a4a2054377ebf9d4b1ce

comment:10 Changed 6 years ago by Vickie Lynch

same answer and 10% faster

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10284

Note: See TracTickets for help on using tickets.