Skip to content

Update transform schemas to support transform-1.4.0 schema#568

Merged
Cadair merged 5 commits intoDKISTDC:mainfrom
braingram:new_transforms
Jul 1, 2025
Merged

Update transform schemas to support transform-1.4.0 schema#568
Cadair merged 5 commits intoDKISTDC:mainfrom
braingram:new_transforms

Conversation

@braingram
Copy link
Contributor

@braingram braingram commented May 22, 2025

A new version of asdf-transform-schemas will soon be released with a new transform-1.4.0 schema. See asdf-format/asdf-transform-schemas#120 for more details on the motivation for the new schema (tldr; this decouples the transform schemas from the core schemas in the standard to allow easier fixes for things like the ndarray mask bug: asdf-format/asdf#1909).

This PR updates the dkist wcs schemas to use the new transform-1.4.0 schema.

A related asdf-astropy PR (astropy/asdf-astropy#279) includes checks to verify the updates don't break the current dkist release due to a sort of self-referencing issue with the now old transform-1.3.0 schema. That PR makes the assumption that the dkist changes in this PR will end up in manifest 1.5.0.

This PR is part of an effort to untangle some dependencies in transform schemas. See asdf-format/asdf-wcs-schemas#68 for probably too much context.

@codspeed-hq
Copy link

codspeed-hq bot commented May 22, 2025

CodSpeed Performance Report

Merging #568 will not alter performance

Comparing braingram:new_transforms (cbca929) with main (731e7f0)

Summary

✅ 12 untouched benchmarks

@braingram braingram marked this pull request as ready for review May 28, 2025 13:59
@braingram
Copy link
Contributor Author

The py313 failure was due to this PR being updated too soon after the asdf-transform-schemas release. I suspect re-triggering that run will allow it to succeed (but I don't seem to have that button).

@braingram
Copy link
Contributor Author

@Cadair I'm starting the process of releasing asdf-astropy with the version check for the manifest version introduced in this PR. If there are plans to publish a different dkist-wcs-1.5.0 manifest let me know.

@Cadair
Copy link
Member

Cadair commented Jun 11, 2025

We are likely to do a new manifest or two soon, but you are welcome to 1.5 😆

@Cadair Cadair added the Run downstream CI Run's the downstream CI workflow on a PR label Jun 12, 2025
Copy link
Member

@Cadair Cadair left a comment

Choose a reason for hiding this comment

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

lgtm

@Cadair Cadair merged commit 24ba3b9 into DKISTDC:main Jul 1, 2025
27 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Run downstream CI Run's the downstream CI workflow on a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants