Skip to content

Expose model_transformation tag from GeoTIFF IFDs#75

Merged
maxrjones merged 3 commits intovirtual-zarr:mainfrom
jakenotjay:jake/get-model-transform
Feb 26, 2026
Merged

Expose model_transformation tag from GeoTIFF IFDs#75
maxrjones merged 3 commits intovirtual-zarr:mainfrom
jakenotjay:jake/get-model-transform

Conversation

@jakenotjay
Copy link
Contributor

  • Add model_transformation to the list of extra IFD keys extracted in _get_attributes, so GeoTIFFs that use a 4x4 affine transformation matrix (ModelTransformationTag) instead of the more common ModelPixelScale + ModelTiepoint pair have their georeferencing info available in the dataset attributes.
  • Add a network test against an AEF embedding TIFF that uses this tag, verifying the transformation is present and the scale/tiepoint tags are absent.

@jakenotjay
Copy link
Contributor Author

@maxrjones can I do anything about this failed "test-min-versions-build" error, or is this expected?

@maxrjones
Copy link
Member

| @maxrjones can I do anything about this failed "test-min-versions-build" error, or is this expected?

hey @jakenotjay, thanks for the PR. I switched away from dasghub/dvc in #76 to avoid these errors. All should work if you merge main into your branch

@jakenotjay
Copy link
Contributor Author

@maxrjones awesome, thanks for that!

Copy link
Member

@maxrjones maxrjones left a comment

Choose a reason for hiding this comment

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

Thank you!

@maxrjones maxrjones merged commit f9aeb06 into virtual-zarr:main Feb 26, 2026
5 checks passed
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