[INSTRM-1789] Add customizable kernel sizes to centroiding Created: 14/Nov/22 Updated: 04/Jan/23 Resolved: 04/Jan/23 |
|
| 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: | CanBeClosed, EngRun | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Story Points: | 3 |
| Sprint: | PreEngRun09Dec |
| Description |
|
Add the ability to increase the size of the convolution kernel for sextractor, for unfocussed images. |
| Comments |
| Comment by karr [ 15/Nov/22 ] |
|
An implementation has been pushed; this will need to be tested in the simulator, particularly as the calling in AgcCmd.py has been changed (centroid parameters are initialized at the start, and updated by calling setCentroidParams directly). A fwhm field has been added to the cParms dictionary which is None unless otherwise specified. |
| Comment by karr [ 01/Dec/22 ] |
|
Work during the commissioning run shows that tweaking the blending parameter was more useful than the kernel size; and required less fine tuning of other parameters. Consequently, this branch was used for that implementation, which was merged during the commissioning run. |
| Comment by hassan [ 04/Jan/23 ] |
|
Already merged to master. |