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.

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

Convert step-30 from using eps/gnuplot to svg/vtu. #9493

Merged
merged 1 commit into from
Feb 12, 2020

Conversation

bangerth
Copy link
Member

@bangerth bangerth commented Feb 7, 2020

The figures as currently used are actually quite ugly. We can do
substantially better. While there, also clean up the
function that creates output a bit.

I've already put the figures on the website.

In reference to #5982.

Copy link
Contributor

@GrahamBenHarper GrahamBenHarper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've already put the figures on the website.

I wanted to double-check the mesh outputs but I probably don't have the permissions to view them. I don't know if that's intentional or not.
https://www.dealii.org/images/steps/developer/step-30.grid-0.aniso.9.1.png

Other than that everything looks great.

The figures as currently used are actually quite ugly. We can do
substantially better. While there, also clean up the
function that creates output a bit.
@bangerth
Copy link
Member Author

bangerth commented Feb 7, 2020

Ah, good check -- these file names should all have had "9.2" instead of "9.1" in the names. I've fixed the patch. Thanks!

@bangerth
Copy link
Member Author

Ping?

Copy link
Member

@masterleinad masterleinad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@masterleinad
Copy link
Member

/rebuild

@kronbichler kronbichler merged commit 9398a7d into dealii:master Feb 12, 2020
@bangerth bangerth deleted the 30 branch February 12, 2020 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants