XMLWordPrintable

#### Details

• Type: Story
• Status: Done
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
None
• Story Points:
1
• Team:
Data Release Production

#### Description

Implement RFC-306.

#### Activity

No builds found.
Paul Price created issue -
Field Original Value New Value
Link This issue is triggered by RFC-306 [ RFC-306 ]
Hide
Paul Price added a comment -

And add srcMatchFull and deepCoadd_measMatchFull to the obs packages that don't have it.

Show
Paul Price added a comment - And add srcMatchFull and deepCoadd_measMatchFull to the obs packages that don't have it.
Hide
Paul Price added a comment -

This has passed Jenkins (on the one system that didn't need to update ci_hsc during the Nebula downtime).

Show
Paul Price added a comment - This has passed Jenkins (on the one system that didn't need to update ci_hsc during the Nebula downtime).
Hide
Paul Price added a comment -

Thanks for agreeing to review this, Maria Patterson [X]. I think you'll find it pretty straight-forward and repetitive, but mistakes may have slipped in so I appreciate your looking it over.

Show
Paul Price added a comment - Thanks for agreeing to review this, Maria Patterson [X] . I think you'll find it pretty straight-forward and repetitive, but mistakes may have slipped in so I appreciate your looking it over.
 Reviewers Maria Patterson [ mtpatter ] Status To Do [ 10001 ] In Review [ 10004 ]
Hide
Maria Patterson [X] (Inactive) added a comment -

It looks like deepCoadd_srcMatch is still in two places in lsst-dm/pegasus_workflow, just searching Github. Do you want to do PRs for that too, if it should be updated? (Or make a ticket?)

Show
Maria Patterson [X] (Inactive) added a comment - It looks like deepCoadd_srcMatch is still in two places in lsst-dm/pegasus_workflow, just searching Github. Do you want to do PRs for that too, if it should be updated? (Or make a ticket?)
Hide
Paul Price added a comment -

I know nothing about pegasus_workflow. Maybe Hsin-Fang Chiang does?

Show
Paul Price added a comment - I know nothing about pegasus_workflow . Maybe Hsin-Fang Chiang does?
Hide
Hsin-Fang Chiang added a comment -

Yes, I will update pegasus_workflow. Thanks for informing me!

Show
Hsin-Fang Chiang added a comment - Yes, I will update pegasus_workflow. Thanks for informing me!
Hide
Hsin-Fang Chiang added a comment -

I added a corresponding PR in the pegasus_workflow and will merge it after other packages are merged.

Show
Hsin-Fang Chiang added a comment - I added a corresponding PR in the pegasus_workflow and will merge it after other packages are merged.
Hide
Maria Patterson [X] (Inactive) added a comment -

Ok, looks good. I got half way through commenting that the file template names were not updated before I realized that some of the PRs note that that is intentional. (Sorry!!!) Are they all intentionally left that way? Just triple checking. If that's all good for you, looks good by me.

Show
Maria Patterson [X] (Inactive) added a comment - Ok, looks good. I got half way through commenting that the file template names were not updated before I realized that some of the PRs note that that is intentional. (Sorry!!!) Are they all intentionally left that way? Just triple checking. If that's all good for you, looks good by me.
Maria Patterson [X] (Inactive) made changes -
 Status In Review [ 10004 ] Reviewed [ 10101 ]
Hide
Paul Price added a comment -

Thank you Maria Patterson [X]!

I didn't touch the file templates on purpose, because I don't want to invalidate any of the existing data on disk. If people want to change them later, I'd support it, but it would take some organisation.

Show
Paul Price added a comment - Thank you Maria Patterson [X] ! I didn't touch the file templates on purpose, because I don't want to invalidate any of the existing data on disk. If people want to change them later, I'd support it, but it would take some organisation.
Hide
Paul Price added a comment -

Merged to master.

Show
Paul Price added a comment - Merged to master.
 Resolution Done [ 10000 ] Status Reviewed [ 10101 ] Done [ 10002 ]
 Component/s ci_hsc [ 13140 ]
 Link This issue blocks DM-9907 [ DM-9907 ]

#### People

Assignee:
Paul Price
Reporter:
Paul Price
Reviewers:
Maria Patterson [X] (Inactive)
Watchers:
Hsin-Fang Chiang, Maria Patterson [X] (Inactive), Paul Price
0 Vote for this issue
Watchers:
3 Start watching this issue

#### Dates

Created:
Updated:
Resolved:

#### Jenkins

No builds found.