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

PyPlot and Gadfly to Plots #67

Closed
Nosferican opened this Issue Sep 6, 2018 · 5 comments

Comments

Projects
None yet
4 participants
@arnavs

This comment has been minimized.

Collaborator

arnavs commented Sep 6, 2018

As discussed, this is a good first issue for @ajozefiak. After you're done getting up to speed, maybe give this a go, and ping me if there are any issues.

@arnavs arnavs self-assigned this Sep 6, 2018

@arnavs

This comment has been minimized.

Collaborator

arnavs commented Sep 6, 2018

Until Adam joins the repo, I'll self-assign these just to keep track and remind myself to stay apprised of their status.

@arnavs arnavs referenced this issue Sep 6, 2018

Closed

Ensure Full Test Coverage #70

3 of 5 tasks complete

@Nosferican Nosferican changed the title from Plots to PyPlot to Plots Sep 11, 2018

@Nosferican Nosferican changed the title from PyPlot to Plots to PyPlot and Gadfly to Plots Sep 11, 2018

@Nosferican Nosferican assigned ajozefiak and unassigned arnavs Sep 13, 2018

@ajozefiak

This comment has been minimized.

Collaborator

ajozefiak commented Sep 30, 2018

I've gotten stuck with the surface plot in "Job Search IV: Modeling Career Choice". I am not sure if Plots has enough 'control' over the surface plot as compared to using GR or PyPlot directly.

I've also gotten stuck on the last snippet of code in the Globalization and Cycles lecture. I've pushed a new branch matsuyama_plots where all of the plots except for the last one have been converted to the Plots API. Specifically, the last plot is an interactive plot and perhaps someone else knows how to work with this better than me.

@jlperla

This comment has been minimized.

Collaborator

jlperla commented Sep 30, 2018

@ajozefiak Thanks so much on that. We can review it next week. If we need to drop some of the features, it may be worth it for reproducibility of setups.

@arnavs arnavs assigned arnavs and unassigned ajozefiak Oct 2, 2018

@arnavs

This comment has been minimized.

Collaborator

arnavs commented Oct 2, 2018

Handling these last two plots after discussion with @ajozefiak, since they're special cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment