Skip to content

Separate r-base image and tested#192

Open
grallewellyn wants to merge 3 commits intodevelopfrom
separate-r-base
Open

Separate r-base image and tested#192
grallewellyn wants to merge 3 commits intodevelopfrom
separate-r-base

Conversation

@grallewellyn
Copy link
Member

Both images will be published
Can test now with mas.dit.maap-project.org/root/maap-workspaces/2i2c/r-base:develop and mas.dit.maap-project.org/root/maap-workspaces/2i2c/r:develop

@wildintellect
Copy link
Contributor

@grallewellyn can you clarify which image is for what purpose?

@grallewellyn
Copy link
Member Author

mas.dit.maap-project.org/root/maap-workspaces/2i2c/r-base:develop is the base image for VEDA and mas.dit.maap-project.org/root/maap-workspaces/2i2c/r:develop has MAAP extensions and is built from r-base

- nodefaults
dependencies:
# Python packages only - NO R packages in conda
- gdal=3.10.2
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you put a comment here indicated why we are pinning this version of gdal from conda?

Copy link
Member Author

Choose a reason for hiding this comment

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

That is because it was the last version the R image was built with and we pin our packages in our environment.ymls for reproducibility
Should I upgrade gdal?

Copy link
Contributor

Choose a reason for hiding this comment

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

That seems ok if the idea is to just match the conda gdal to the same version and the system gdal (that R uses). I do anticipate keeping track of that being a hassle later.


RUN conda init

RUN awscliv2 --install
Copy link
Member Author

Choose a reason for hiding this comment

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

This fix did not work and I need to debug if we want to keep awscliv2, can resolve in another PR along with rgee bug

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.

2 participants

Comments