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.

Sign up for GitHub

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

Jump to bottom

Improve resolution in step 8. #8539

Merged
merged 1 commit into from Aug 13, 2019
Merged

Improve resolution in step 8. #8539

merged 1 commit into from Aug 13, 2019

Conversation

chronictectonic
Copy link
Contributor

Improved resolution to generate better looking figures (attached here).
step-8 x
step-8 y

@drwells
Copy link
Member

drwells commented Aug 9, 2019

Looks good! Would you please crop the images so that we do not see the coordinates axes?

@chronictectonic
Copy link
Contributor Author

Thanks! I noticed that shortly after uploading.
step-8 x
step-8 y

@chronictectonic
Copy link
Contributor Author

And the vector field
step-8 vectors

@bangerth
Copy link
Member

This increases run-time of this program from 4 to 40 seconds -- which seems entirely reasonable.

@chronictectonic -- do you think you could come up with a small changelog entry that, in one line, explains what you changed? Let us know once you've got it added and we can merge!

@bangerth
Copy link
Member

I've updated the pictures on the website with the ones from here.

@bangerth
Copy link
Member

I've used the pictures in #8557, by the way. Thanks for creating them!

@bangerth bangerth merged commit 79b373a into dealii:master Aug 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bangerth bangerth bangerth approved these changes

Assignees
No one assigned
Labels
Tutorials
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@chronictectonic @drwells @bangerth