-
Notifications
You must be signed in to change notification settings - Fork 47
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
Easier local setup of mrfgen tools #185
Conversation
env was unsolvable as it was, the gdal pin restricting to earlier pillow/rasterio rasterio 1.3.10 wanted gdal 3.9 with the current gdal pin, I got - pillow 10.0.0 - rasterio 1.3.7
the install info doesn't need to be in separate file and wouldn't work anyway since there is no requirements.txt in this directory the example at the end also doesn't seem to exist anymore
Thanks for the contribution! We'll run some test with GIBS to make sure our builds are happy. We've had some prior discussions about making mrfgen its own standalone repository. I'd be interested to hear if that would be a worthwhile pursuit. |
Yes I did also think about this. It would be convenient if it had a conda-forge recipe, say. And that would be much easier to set up if mrfgen had its own repo. I was also curious about the status of https://github.com/nasa-gibs/gpu-data-to-image |
It was intended as a prototype and has been dormant for a few years. It worked great back then and could be revived. We wanted to integrate it with https://github.com/apache/sdap-nexus but never had the cycles to put things together. |
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.
Thank you!
The main change is the addition of a Conda env spec and some edits to the mrfgen readme.
I also wrote a small install script and made some minor changes in order for
--help
to work without error for all of the tools.src/mrfgen/RgbPngToPalPng.py
had some\r
s in it, I randos2unix
on it to get it to work as an executable. I also linkedoe_utils.py
since it's needed.I wasn't able to get all the same versions from conda-forge as pinned elsewhere, so open to thoughts about that.