add ottr_viz image#15
Open
kweav wants to merge 3 commits into
Open
Conversation
Contributor
Author
|
Right now this is pushed to dockerhub as |
Contributor
|
you could add yourself as the maintainer @kweav |
added FH email for maintainer
Contributor
Author
|
@carriewright11 thanks for approving this! Going to finish the example graphs for the data viz course before I merge in case I need any more packages |
Contributor
makes sense! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Making a data visualization ottr docker image that has common plotting libraries for both R and Python. Building the data visualization course, I was needing to install packages and got to the point where just making the container was definitely easier/worth it.
Type of change
Built using ottr_python as my base image. That has everything base_ottr has plus a lot of common python packages. This adds visualization packages, especially those we've used for workshops or making example figures/content in the data viz talk. Note that it's adding visualization packages for both R and Python. Had to install several dependencies before I could install the visualization packages -- did some troubleshooting with copilot to figure out what dependencies were needed whenever building failed.
How Has This Been Tested?
I built this docker image locally then ran the container, using a volume to connect it to the data visualization course files. I ran
quarto render 07_wrapping_up.qmdto verify that it successfully rendered my content the way that I expected/wanted.One other note:
I edited the pull request template so that it has a checklist for what you need to do when you create a new Docker image (using the instructions in the README), because I keep looking at the README whenever I need to make a new image