Ticket #9441 (closed: fixed)
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: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
