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

Add a few references to the FE_Q_iso_Q1 element documentation. #12765

Merged
merged 1 commit into from Sep 18, 2021

Conversation

bangerth
Copy link
Member

No description provided.

@bangerth
Copy link
Member Author

/rebuild

Copy link
Member

@kronbichler kronbichler 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, thank you for the references. I added just a small comment below.

journal = {Journal of Scientific Computing}
}

@article{taho73,
Copy link
Member

Choose a reason for hiding this comment

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

Can we rename this into

Suggested change
@article{taho73,
@article{Taylor73,

to give those people who do not want to click the reference an idea what the paper is about? (I do not like to click on reference links and just look at the name of the web page linked to, which picks up this reference and thus I like expressive names for the papers.)

@bangerth
Copy link
Member Author

So changed! :-)

@kronbichler kronbichler merged commit aa434d3 into dealii:master Sep 18, 2021
@kronbichler
Copy link
Member

Thanks!

@bangerth bangerth deleted the references branch September 20, 2021 03:40
@bangerth
Copy link
Member Author

I tend to think that we ought to have more references to the literature, and this one was a fun find while working on a project in which the Taylor-Hood element was important.

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

2 participants