[PIPE2D-506] Migrate code to use LSST gen3 middleware Created: 07/Jan/20 Updated: 26/May/22 Resolved: 26/May/22 |
|
| Status: | Done |
| Project: | DRP 2-D Pipeline |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| 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 | ||
| Attachments: |
|
||||||||||||||||||||
| Issue Links: |
|
||||||||||||||||||||
| Story Points: | 4 | ||||||||||||||||||||
| Sprint: | 2DDRP-2022 B, 2DDRP-2022 C, 2DDRP-2022 D | ||||||||||||||||||||
| Reviewers: | hassan | ||||||||||||||||||||
| Description |
|
Following |
| Comments |
| Comment by price [ 05/Mar/20 ] |
|
This ticket should be blocked on another ticket to upgrade the LSST base stack. Because a ready Gen3 middleware is not yet available, and Gen2 will be supported for at least another year, I suggest we needn't think about this ticket for another six months. The right time to think about this will be when LSST start using Gen3 for their fortnightly RC2 runs. |
| Comment by price [ 05/Mar/22 ] |
|
Thought: the Gen3 butler doesn't care about filename templates. We're going to need a facility to extract files from the butler and write them with the correct filename template for distribution. |
| Comment by price [ 03/May/22 ] |
|
I've got the entire calib and science pipelines working under Gen3. There is more work that needs to be done, but having this merged will allow us to work on the below tasks one by one: This effectively replaces |
| Comment by hassan [ 07/May/22 ] |
|
Reviewed pull requests related to this ticket. Changes look fine. Also looked at the pfsMerged output from an example SuNSS visit (71280) processed with the tickets\ |
| Comment by rhl [ 07/May/22 ] |
|
I'd have expected the results to be equal to close to machine precision. This needs to be investigated, but doing so as part of PIPE2D-1031 is fine. |
| Comment by price [ 07/May/22 ] |
|
My first guess is that there are configuration parameters that differ between the pipelines. This is exactly what PIPE2D-1031 was created for. We will not drop Gen2 support until we are convinced the Gen3 pipeline is performing at the level of Gen2. |
| Comment by price [ 10/May/22 ] |
|
Merging this work soon is important, since there are a bunch of changes that I don't want to conflict with other work. On the other hand, we don't want to merge such a large change right before the observing run on the off chance that it causes problems. I propose to merge immediately after the observing run, and in the mean time base other non-urgent work off of this ticket branch. |
| Comment by price [ 26/May/22 ] |
|
Merged. |