-
Notifications
You must be signed in to change notification settings - Fork 31
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
Plots replace PyPlot in TwoDNavierStokes examples #73
Conversation
@@ -25,13 +25,13 @@ nothing # hide | |||
# | |||
# First, we pick some numerical and physical parameters for our model. | |||
|
|||
n, L = 128, 2π # grid resolution and domain length | |||
n, L = 256, 2π # grid resolution and domain length |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😏
nsteps = 4000 # total number of steps | ||
nsubs = 1000 # number of steps between each plot | ||
dt = 5e-3 # timestep | ||
nsteps = 8000 # total number of steps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This may be a pretty heavy example for docs. But we'll see how long they take to build; maybe it will be fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
128 would work OK also... but 256 looks better.
The problem will arise when we make all examples producing 256^2 animations...
Codecov Report
@@ Coverage Diff @@
## master #73 +/- ##
=======================================
Coverage 98.62% 98.63%
=======================================
Files 6 6
Lines 510 511 +1
=======================================
+ Hits 503 504 +1
Misses 7 7
Continue to review full report at Codecov.
|
This is the beginning. We should ultimately drop PyPlot from docs/examples whatsoever.