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 bib entry for dynamic core reference Nimmo 2004 #2319

Merged
merged 1 commit into from Jun 20, 2018

Conversation

jonpkay
Copy link
Contributor

@jonpkay jonpkay commented Jun 19, 2018

Fixes #2238

@gassmoeller
Copy link
Member

/run-tests

Copy link
Contributor

@bangerth bangerth left a comment

Choose a reason for hiding this comment

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

I think this is reasonable. In the comments in the source code, \cite{...} doesn't actually do anything useful, but having a reference with a discoverable name is not a bad idea either.

OK to merge if the tester is happy.

Copy link
Member

@gassmoeller gassmoeller 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. One minor comment, then this is good to go.

@@ -160,16 +160,16 @@ namespace aspect
{
prm.declare_entry ("Tm0","1695",
Patterns::Double (0),
"Melting curve (Nimmo et al. [2004] eq. (40)) parameter Tm0. Units: $K$.");
"Melting curve (\cite{NPB+04} eq. (40)) parameter Tm0. Units: $K$.");
Copy link
Member

Choose a reason for hiding this comment

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

You used the double backslash \\cite for the description above, but not in this case and the three occurences below. Please add it in those places as well. Just to explain: Everything in the double quotation marks is parsed through the deal.II parameter handler, and thus needs an extra escape character. That is why it is fine to write \cite in the documentation of the header file, but you need to write \\cite for every parameter description in the .cc file.

Copy link
Contributor

@bangerth bangerth left a comment

Choose a reason for hiding this comment

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

@gassmoeller had requested changes, so I'll block for a moment. Talk to me if you need help!

@tjhei tjhei merged commit 733aee4 into geodynamics:master Jun 20, 2018
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