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

OHW22 R environment changes #49

Open
abkfenris opened this issue Jul 28, 2022 · 26 comments
Open

OHW22 R environment changes #49

abkfenris opened this issue Jul 28, 2022 · 26 comments

Comments

@abkfenris
Copy link
Collaborator

Please comment with any R packages that are need for OHW22 here. We are looking to get all of these at least a week before the main event starts, so we aren't changing anything for the undergrads!

Please use a checkbox for each package so we can track what has been added, ex:

- [ ] some-awesome-package, link to documentation if the conda package isn't obvious to someone who doesn't use it, or CRAN source

The current R environment is defined in https://github.com/oceanhackweek/jupyter-image/blob/main/r/environment.yml

@btupper
Copy link

btupper commented Aug 2, 2022

@abkfenris
Copy link
Collaborator Author

abkfenris commented Aug 5, 2022

@abkfenris
Copy link
Collaborator Author

@btupper those should be deployed.

I was able to install most via Conda, except for the few from CRAN that I left unchecked. When an install from CRAN or Github fails, it doesn't exit correctly to fail the build, so I'd suggest trying the environment to see if everything is working as you expect.

@btupper
Copy link

btupper commented Aug 5, 2022

OK! Thank you! @jevanilla and @Cross22 - can you each give it a shakedown?

@ocefpaf
Copy link
Member

ocefpaf commented Aug 10, 2022

We start next Monday! It would be awesome if you all can upload your notebooks to our tutorial repo and test them in the hub.

@btupper
Copy link

btupper commented Aug 11, 2022

Roger that! We shall be using the RStudio server and Rmarkdown documents for the tutorials.

The first tutorial is being vetted within the lab to catch the most egregious boo-boos. The first kicks off with a short video which is linked in the Rmarkdown tutorial. We are assuming that the presenter can simply play the youtube from that link - happy to be corrected. Hollywood has yet to call me, and I think I know why now... https://youtu.be/2ZWFYWMwq1c

The second tutorial is under construction but should be ready in the next 2 days. It uses the same software as the first tutorial.

We are juggling vacations and a small personnel shuffle this week, but, have no worries, we'll make it. What could possibly go wrong?

P.S. The Rmarkdown files are located in the xytz R package. https://github.com/BigelowLab/xyzt

@jevanilla
Copy link

jevanilla commented Aug 12, 2022

abkfenris added a commit to abkfenris/jupyter-image that referenced this issue Aug 12, 2022
@abkfenris
Copy link
Collaborator Author

Is there a way to get remotes:install_cran and remotes:install_github to exit non 0?

https://github.com/oceanhackweek/jupyter-image/runs/7813411273?check_suite_focus=true#step:6:2900

It really doesn't want to let me know when something fails and breaks the build.

I'll take another swing with the dependencies it wants.

@abkfenris
Copy link
Collaborator Author

Ok, within 6 hours 2i2c will get a PR with the updated image, then we need to wait for one of their engineers to merge it.

@jevanilla
Copy link

We start next Monday! It would be awesome if you all can upload your notebooks to our tutorial repo and test them in the hub.

@ocefpaf I sent a PR with my notebooks to oceanhackweek/ohw-tutorials

oceanhackweek/ohw-tutorials#87

@abkfenris
Copy link
Collaborator Author

Those changes should be live on the hub: 2i2c-org/infrastructure#1622

@abkfenris
Copy link
Collaborator Author

abkfenris commented Aug 13, 2022

It looks great, thanks!! As I mentioned over email, it looks like we don't yet have a couple of the libraries in our R image. But we'll manage. For reference, the libraries seem to be:

  • emo
  • markdown (though we do have rmarkdown)
  • metR
  • udunits2
  • xaringan

Gracias!

Updated: Note to self (I'll move these notes elsewhere, soon): All these packages except emo are in conda-forge.

Originally posted by @emiliom in oceanhackweek/ohw-tutorials#91 (comment)

@abkfenris
Copy link
Collaborator Author

emo can't be installed: #55

@emiliom
Copy link
Member

emiliom commented Aug 14, 2022

emo can't be installed: #55

Based on @paocorrales's R code and some Googling, I'm pretty sure it's this emoji library:
https://github.com/hadley/emo

And apparently it's installed from github. In R speak:

# install.packages("devtools")
devtools::install_github("hadley/emo")

Thanks for adding the R packages from conda-forge!

@abkfenris
Copy link
Collaborator Author

Ok, that looks like it did the trick.

@emiliom
Copy link
Member

emiliom commented Aug 14, 2022

Thank you, @abkfenris & @ocefpaf !!

ref PR #55 and PR oceanhackweek/ohw-tutorials#91

@emiliom
Copy link
Member

emiliom commented Aug 15, 2022

This is not a final request yet, but I wanted to draft it here. It's for a tutorial on Wednesday.

The instructor said tmap is used (plus other packages already in the R image), and I've asked her to check and let us know if tmaptools is also needed, so we can add both in one PR.

That rounds out the packages needed for the 4 tutorials in the Espanol satellite, R & Python.

@abkfenris
Copy link
Collaborator Author

Tmap and friend is in #58

@emiliom
Copy link
Member

emiliom commented Aug 16, 2022

Darn you're too fast! I was about to add one LAST one (I swear):

  • mapproj. It's on conda-forge.

@abkfenris
Copy link
Collaborator Author

Looks like something else already brought mapproj in.

https://conda.anaconda.org/conda-forge/linux-64/r-mapproj-1.2.8-r41hcfec24a_0.tar.bz2#452f9179c0b907ec358cc6b33ef0d2b5

@emiliom
Copy link
Member

emiliom commented Aug 16, 2022

Awesome! I forgot to check the conda lock file.

@loreabad6
Copy link

Hello, not sure if it will be needed in the end but can you please add "sfnetworks", "tidygraph" and "igraph"

@loreabad6
Copy link

Hello, not sure if it will be needed in the end but can you please add "sfnetworks", "tidygraph" and "igraph"

Hi again, can you let me know if the packages could be installed.
If still on time can you add "mapview" as well?

As a backup I will add the demo to my local binder. Thanks!

@abkfenris
Copy link
Collaborator Author

Hello, not sure if it will be needed in the end but can you please add "sfnetworks", "tidygraph" and "igraph"

Hi again, can you let me know if the packages could be installed. If still on time can you add "mapview" as well?

As a backup I will add the demo to my local binder. Thanks!

I'm working on adding those now. The image is currently building https://github.com/oceanhackweek/jupyter-image/actions/runs/2876389472
Then I'll submit the new image to 2i2c to get them to approve it.

@abkfenris
Copy link
Collaborator Author

2i2c has approved it, now we are waiting on one of their workflows to finish, then the packages will be usable once you shut down and restart your server. https://github.com/2i2c-org/infrastructure/runs/7882280892?check_suite_focus=true

@emiliom
Copy link
Member

emiliom commented Aug 17, 2022

Wow, thank you @abkfenris !

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

6 participants