Skip to content

ggplot#180

Merged
wrightaprilm merged 6 commits intodatacarpentry:gh-pagesfrom
tmorrell:ggplot
Apr 24, 2017
Merged

ggplot#180
wrightaprilm merged 6 commits intodatacarpentry:gh-pagesfrom
tmorrell:ggplot

Conversation

@tmorrell
Copy link
Copy Markdown
Contributor

Include install instructions for ggplot and improve formatting

@wrightaprilm
Copy link
Copy Markdown
Contributor

Hi Thomas, having a quick look at this. Mostly looks good, but I was wondering if bokeh is required by ggplot? I was under the impression that it is not. If that's not the case, we can have the learners install it, but if it's not required, let's not include it.

Also, about removing the inline statement. I'm fine with that, how would you feel about including %matplotlib notebook, which allows for some handy plot utilities, like zooming on plots or exporting though point-and-click?

@tmorrell
Copy link
Copy Markdown
Contributor Author

At least on my current anaconda install ggplot is not present in the main channel (conda install ggplot doesn't work). The -c on the install line refers to the bokeh channel and will only install ggplot. There might be a better option though?

Thanks for the %matplotlib notebook suggestion-I've added it to the PR.

@wrightaprilm wrightaprilm merged commit c5a29e9 into datacarpentry:gh-pages Apr 24, 2017
@wrightaprilm
Copy link
Copy Markdown
Contributor

Aha, I see the issue. Well, this looks good to me; thank you! Merging now.

@tmorrell tmorrell deleted the ggplot branch April 25, 2018 16:54
zkamvar pushed a commit that referenced this pull request May 8, 2023
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