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

actinia-ubuntu: multi-stage build oriented on Alpine #61

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

ninsbl
Copy link
Contributor

@ninsbl ninsbl commented Sep 25, 2023

For the DeepLearning framework torch Alpine is not a supported distribution, so in order to run that, one has to use Ubuntu (or similar) images...

This PR attempts to update the Ubuntu Docker image to 22.04 and to improve both size and content, amongst others using multi-stage build.

SNAP is updated to version 9, but snappy support is dropped.

In any case, this:
OSGeo/grass#3170
should be merged and backported to 8.3 first for this to build properly...

Copy link
Member

@mmacata mmacata left a comment

Choose a reason for hiding this comment

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

Thanks, very cool!

Is snap without snappy helpful in this docker setup? If yes, why are the files related to the snap installation moved? If there is a reason, it would be nice to have the others removed to not have them twice.

@@ -1,48 +1,43 @@
FROM osgeo/grass-gis:stable-ubuntu
FROM osgeo/grass-gis:stable-ubuntu as grass
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
FROM osgeo/grass-gis:stable-ubuntu as grass
FROM osgeo/grass-gis:current-ubuntu as grass

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I do have some changes I have not pushed yet (as some stuff was failing).
The SNAP stuff could be more separated. Unfortunately, snappy is still not possible to install with Python 3.10.
It will be renamed to esa_snappy and is probably first usable with SNAP 10 (just guessing). Will push an update soon...

@ninsbl
Copy link
Contributor Author

ninsbl commented Sep 28, 2023

For me personally, SNAP is still useful, also without snappy. esa_snappy is around the corner, but unfortunately, still not yet working with Python > 3.10. Manually building the JPY java-python-bridge does not help either.
Here is some more infor on esa_snappy with SNAP 10, but SNAP 10 has not been released...

Files for the SNAP installation are now consolidated...

Remaining thing is now maybe, wait for OSGeo/grass#3170 to be merged and for an updated Docker image fro GRASS GIS to be built and to become available on docker hub...

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.

3 participants