Skip to content

Fix GetTemplateTask error#24

Closed
jeffcarlin wants to merge 2 commits into
mainfrom
u/jcarlin/patch_NB03c
Closed

Fix GetTemplateTask error#24
jeffcarlin wants to merge 2 commits into
mainfrom
u/jcarlin/patch_NB03c

Conversation

@jeffcarlin
Copy link
Copy Markdown
Collaborator

Instead of "coaddExposureHandles", GetTemplateTask now expects "coaddExposures".

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Copy Markdown
Collaborator

@athornton athornton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you want to clear the output before merging, don't you?

The fix itself looks correct to me.

Copy link
Copy Markdown
Collaborator

@athornton athornton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jeffcarlin
Copy link
Copy Markdown
Collaborator Author

Closing without merging because this fix will be done on Douglas Tucker's ticket for the v29.1 recommended bump.

@jeffcarlin jeffcarlin closed this Jun 13, 2025
@jeffcarlin jeffcarlin deleted the u/jcarlin/patch_NB03c branch June 13, 2025 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants