Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Sentinel 1 GRD rendering #90

Merged
merged 3 commits into from
May 13, 2022
Merged

Fix Sentinel 1 GRD rendering #90

merged 3 commits into from
May 13, 2022

Conversation

lossyrob
Copy link
Contributor

@lossyrob lossyrob commented May 13, 2022

Sentinel 1 GRD uses GCPs and does not have a CRS defined. This breaks the standard COGReader.

This commit adds the changes that exist in titiler's GCPCOGReader for Sentinel 1 GRD, and sets the MosaicSTACReader to use the CustomCOGReader.

How Has This Been Tested?

Locally against the item and mosaic endpoints. Tested io-lulc to ensure it doesn't interfere with non-GRD formats. Further testing will be done on deployed tilers

Checklist:

Please delete options that are not relevant.

  • I have performed a self-review
  • Changelog has been updated
  • Unit tests pass locally (./scripts/test)
  • Code is linted and styled (./scripts/format)

lossyrob added 3 commits May 13, 2022 18:45
Sentinel 1 GRD uses GCPs and does not have a
CRS defined. This breaks the standard
COGReader.

This commit adds the changes that
exist in titiler's  GCPCOGReader
for Sentinel 1 GRD, and sets the
MosaicSTACReader to use the CustomCOGReader.
@lossyrob lossyrob merged commit 55069f7 into main May 13, 2022
@lossyrob lossyrob deleted the fix/rde/grd-tiler branch May 13, 2022 23:02
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.

1 participant