[PIPE2D-344] Implement arm merging Created: 08/Feb/19 Updated: 08/Mar/19 Resolved: 08/Mar/19 |
|
| Status: | Done |
| Project: | DRP 2-D Pipeline |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 6.0 |
| Type: | Story | Priority: | Normal |
| Reporter: | hassan | Assignee: | price |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Story Points: | 2 | ||||||||||||||||
| Epic Link: | Arm merging | ||||||||||||||||
| Sprint: | 2019 B | ||||||||||||||||
| Description |
|
Implement arm merging in the DRP. If this is already implemented, provide a reference to the code under GitHub. Implement unit tests if not done already. |
| Comments |
| Comment by rhl [ 08/Feb/19 ] |
|
I don't think we want a pointer to the code, we want the task written and documented in the user manual. We'll want a command line script that wraps it too. |
| Comment by price [ 09/Feb/19 ] |
|
Arm merging is implemented, and it's documented in the user manual. However, the algorithm is a braindead place-holder which needs to be updated with the code from the old makePfsObject function, or something even better. |
| Comment by hassan [ 08/Mar/19 ] |
|
Following price's comments, this issue is addressed. A separate ticket to implement an improved merging algorithm ( |