Skip to content

Comments

Contribute 2 notebooks on R and Zarr#119

Closed
joheisig wants to merge 10 commits intoeopf-toolkit:mainfrom
joheisig:main
Closed

Contribute 2 notebooks on R and Zarr#119
joheisig wants to merge 10 commits intoeopf-toolkit:mainfrom
joheisig:main

Conversation

@joheisig
Copy link

add two notebook chapters for part 5 (Zarr tools)

  1. access using the Rarr package
  2. access using traditional GDAL-based approaches

other edits

  • R docker file: added 2 packages
  • _quarto.yml: uncommented chapter names

Copy link
Collaborator

@gisromerocandanedo gisromerocandanedo left a comment

Choose a reason for hiding this comment

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

Hello, the devs look great.
Can you update the changes, then I will re-run the tests.

@ciaransweet
Copy link
Contributor

Hey @joheisig - Thanks for the PR, we recently moved the R packages into an environment.yaml: https://github.com/eopf-toolkit/eopf-101/blob/main/deployment/environment.yaml the Dockerfile pulls that in now. Could you rebase your change, move the dependencies to the .yaml file, and i'll kick off the workflows, thanks!

@joheisig joheisig marked this pull request as draft January 23, 2026 11:56
@joheisig joheisig marked this pull request as ready for review January 23, 2026 12:39
@joheisig
Copy link
Author

Hey @ciaransweet , thanks for your help. Hope I managed to do what you asked! :)

@ciaransweet
Copy link
Contributor

Hey @ciaransweet , thanks for your help. Hope I managed to do what you asked! :)

Yep perfect! The failure is on me now, will give you a shout once I've fixed it!

@ciaransweet
Copy link
Contributor

ciaransweet commented Jan 23, 2026

Hey @joheisig sorry, could you rebase again please? There's a conflict on the environment.yaml (that's on me, I re-ordered just to trigger our build)

Feel free to just ignore the incoming change and prefer yours.

@joheisig joheisig mentioned this pull request Feb 5, 2026
@joheisig
Copy link
Author

joheisig commented Feb 5, 2026

Closing in favor of #156

@joheisig joheisig closed this Feb 5, 2026
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