-
Notifications
You must be signed in to change notification settings - Fork 7
feat: add @developmentseed/deck.gl-geotiff #244
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
Conversation
The test is flakey, thanks CSDA 🙈
| } | ||
| }, | ||
| new COGLayer({ | ||
| id: "cog-layer", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fyi: if you ever have more than one layer, they must have different ids here
| "@deck.gl/layers": "^9.2.5", | ||
| "@deck.gl/mapbox": "^9.2.5", | ||
| "@developmentseed/deck.gl-geotiff": "^0.1.0", | ||
| "@developmentseed/deck.gl-raster": "^0.1.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think you strictly need to depend on @developmentseed/deck.gl-raster as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I accidentally added that one first. Doesn't hurt for now.
| }); | ||
|
|
||
| test("CSDA Planet", async () => { | ||
| test.skip("CSDA Planet", async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
did this fail? There are some photometric interpretations I don't support out of the box yet
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Failed for different reasons (CSDA's API being flakey 🙈)
🚀 Great to hear. If you find instances of COGs that don't work out of the box, keep a list/create an issue in deck.gl-raster. In the future, I'd like to have a wrapper library I.e. you should be able to pass in a Sentinel STAC item (or really any item with eo:bands correctly set) and an NDVI or etc preset should work out of the box. |
|
🎉 This PR is included in version 0.13.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
It's wicked fast:
Screen.Recording.2026-01-12.at.12.14.05.PM.mov
cc @kylebarron
Closes #243
Checklist
yarn format)yarn lint)yarn build)yarn test)