[INSTRM-1471] Modify centroiding based on real data Created: 08/Dec/21  Updated: 02/Sep/22  Resolved: 06/May/22

Status: Done
Project: Instrument control development
Component/s: ics_agccActor
Affects Version/s: None
Fix Version/s: None

Type: Story Priority: Normal
Reporter: karr Assignee: karr
Resolution: Done Votes: 0
Labels: EngRun
Σ Remaining Estimate: Not Specified Remaining Estimate: Not Specified
Σ Time Spent: Not Specified Time Spent: Not Specified
Σ Original Estimate: Not Specified Original Estimate: Not Specified

Issue Links:
Duplicate
is duplicated by INSTRM-929 Perform object detection and centroid... Done
Relates
relates to INSTRM-1479 Add flags for region of image in cent... Done
Sub-Tasks:
Key
Summary
Type
Status
Assignee
INSTRM-1472 Modify windowed centroid algorithm fo... Sub-task Won't Fix karr  
INSTRM-1473 Investigate performance of sep for ce... Sub-task Done karr  
INSTRM-1474 Correctly process raw images before c... Sub-task Done karr  
INSTRM-1475 Create configuration file for agcc im... Sub-task Done karr  
INSTRM-1513 Update centroid parameters for sep Sub-task Done karr  
Story Points: 2
Sprint: PreEngRun05 B, PreEngRun05 C, PreEngRun05 D, PreEngRun05 E, PreEngRun05 F, preEngRun07Sep

 Description   

With the acquisition of real data from the agc, we need to modify the centroiding process to make it more robust. 



 Comments   
Comment by karr [ 16/Dec/21 ]

After some work with the images, it is clear we need a robust background subtraction/deblending routine and to carefully  define the clean region of the image before centroiding. Therefore, using sep, which is slower but much more featured, is the best option, as it will handle a wider range of conditions. Splitting the image into two for background subtraction avoids artifacts in the centre of the image; need to check if masking is a better option. 

Comment by karr [ 13/Jan/22 ]

These updates, plus 1479, are pushed  to 1479. Have run the routines on all the data and am checking visually; need to test changes in proper simulation environment before merging. 

Comment by karr [ 17/Mar/22 ]

Routines have been tested in simulation environment; coordinating merges with other brnaches. 

Comment by hassan [ 08/Apr/22 ]

To be closed after INSTRM-1479 branch is merged successfully to master.

Comment by hassan [ 05/May/22 ]

Reopened as sub-task INSTRM-1474 still open

Comment by hassan [ 06/May/22 ]

All sub-tasks are complete. We can close this ticket.

Generated at Sat Feb 10 16:35:50 JST 2024 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.