[INSTRM-1387] Provide plots indicating location of bad cobras Created: 11/Oct/21 Updated: 05/Nov/21 Resolved: 05/Nov/21 |
|
| Status: | Done |
| Project: | Instrument control development |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Normal |
| Reporter: | hassan | Assignee: | karr |
| Resolution: | Done | Votes: | 0 |
| Labels: | EngRun | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Story Points: | 2 |
| Sprint: | PreEngRun4, EngRun3Cleanup |
| Description |
|
Provide plots showing where the bad cobras are located. This should be a utility function that is part of the pfs_utils repo, and making use of the pfs.utils.fiberids.FiberIds class (which in turn uses information in the Grand Fiber Map). Please provide a definition of what is a 'bad' cobra, or whether this is a separate input list. |
| Comments |
| Comment by hassan [ 28/Oct/21 ] |
|
karr: the FiberIds class is available under GitHub: https://github.com/Subaru-PFS/pfs_utils/blob/master/python/pfs/utils/fiberids.py |
| Comment by karr [ 28/Oct/21 ] |
|
routines for plotting cobra convergence diagrams are pushed and merged to dev_ics. example plots can be found at https://sumire.pbworks.com/w/browse/#:~:text=cobraDiagnosticPlotExamples_karr.pdf
|
| Comment by yuki.moritani [ 02/Nov/21 ] |
|
(For recording) Robert and Arnaud send their comments on Slack (commissioning channel). https://sumire-pfs.slack.com/archives/C9TDPMVHT/p1635517080013400
|
| Comment by karr [ 04/Nov/21 ] |
|
Updated, pushed and merged changes based on Robert nad Arnaud's comments. |
| Comment by karr [ 04/Nov/21 ] |
|
The updated file at https://sumire.pbworks.com/w/file/146761128/convergenceResults_sept21_karr.pdf shows the statistics of failed cobras, and a map of poorly performing cobras. Note that these data are taken from runs intended to test other things (like parking next to dots) and shouldn't be over interpreted. |
| Comment by hassan [ 05/Nov/21 ] |
|
Closed based on latest changes to plots and merged code. |