[INSTRM-1412] Integrate multiple centroiding methods in agcc Created: 22/Oct/21 Updated: 28/Oct/21 Resolved: 28/Oct/21 |
|
| Status: | Done |
| Project: | Instrument control development |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Normal |
| Reporter: | karr | Assignee: | karr |
| Resolution: | Done | Votes: | 0 |
| Labels: | EngRun | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Story Points: | 1 | ||||||||||||||||
| Sprint: | EngRun3Cleanup | ||||||||||||||||
| Description |
|
Add back sextractor functionality to the agcc (as is currently done in mcs) for toggling between different centroiding methods. |
| Comments |
| Comment by karr [ 22/Oct/21 ] |
|
Actor is set up with an optional keyword. Setting it to "fast" will use the faster method, setting it to "slow" will use the full sextractor method. No keyword will default to "fast". The format of the output is the same (ie, writing to DB etc), but the definitions of the second moments are different. |
| Comment by hassan [ 27/Oct/21 ] |
|
Jennifer has written all the code, and will push the changes to the ticket branch shortly. |
| Comment by karr [ 28/Oct/21 ] |
|
Checked with Yoshida-san regarding some changes made during the engineering run, compared with changes since commit aabe95c and removed changes which disabled the database writing, but left the updates to hosts and data paths, which have been set for use at Subaru. Merged to master which includes changes made in 1287 and 1412. |
| Comment by hassan [ 28/Oct/21 ] |
|
The changes were merged to master as part of |