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

How do I mimic GeoImage.CopyColorTable in gippy 1.0? #130

Open
ra-tolson opened this issue Aug 30, 2017 · 2 comments
Open

How do I mimic GeoImage.CopyColorTable in gippy 1.0? #130

ra-tolson opened this issue Aug 30, 2017 · 2 comments

Comments

@ra-tolson
Copy link

We're trying to migrate gips from gippy 0.3 to 1.0, and it's not clear how to duplicate GeoImage.CopyColorTable. I can almost mimic it by looping over the bands and doing GeoRaster.add_colortable, but I can't figure out how to get the colortable out of the object I want to copy from. I think GeoRaster._GDALDataSet.GetColorTable() (or similar) would do it, if outside code could call that.

Is there a way to do it that I can't see, or would gippy need to be extended somehow to do this?

@matthewhanson
Copy link
Member

Hello @ags-tolson
When developing gippy 1.0 I ended up removing some functions that weren't used internally by other functions or by algorithms, or that were replaced by other things. I'll review this and if it can't be done currently I'll add the feature back into develop.

Don't hesitate to post any other missing functionality here you come across during the migration, some of it might just need to be features added back in which can be bundled into either a 1.0.1 or 1.1 (if more substantial) release.

@ra-tolson
Copy link
Author

ra-tolson commented Aug 31, 2017

That'd be great, thank you! Overall it's working pretty well for us in terms of doing what gips needs to do; modis is already pretty much converted and landsat is in progress. Mostly I expect to pester you about these little corner cases.

If you're curious about the details, here's a diff which should continue to update:

Applied-GeoSolutions/gips@dev...376-gippy-update

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

No branches or pull requests

2 participants