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

Fix a few bibtex entries. #4931

Merged
merged 1 commit into from Aug 4, 2022
Merged

Fix a few bibtex entries. #4931

merged 1 commit into from Aug 4, 2022

Conversation

bangerth
Copy link
Contributor

/rebuild

@@ -1138,7 +1138,7 @@ @article{STEIN2022229276

@article{https://doi.org/10.1029/2021JB023244,
author = {Zha, Caicai and Lin, Jian and Zhou, Zhiyuan and Xu, Min and Zhang, Xubo},
title = {Effects of Hotspot-Induced Long-Wavelength Mantle Melting Variations on Magmatic Segmentation at the Reykjanes Ridge: Insights From 3D Geodynamic Modeling},
title = {Effects of Hotspot-Induced Long-Wavelength Mantle Melting Variations on Magmatic Segmentation at the {R}eykjanes Ridge: Insights From 3D Geodynamic Modeling},
Copy link
Member

Choose a reason for hiding this comment

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

How do you decide what letters to force uppercase? Ridge is part of the name, shouldn't it also be uppercase? What about 3D and Insights?

Copy link
Contributor

Choose a reason for hiding this comment

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

I do not decide. I take whatever is the "official" export for the publisher is and fix any obvious translation issues of special characters.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I can't say that I had a philosophy, I just wanted to improve things :-) If you tell me what you want, I'll fix it.

It probably doesn't matter much here because we don't use bibtex for these entries, but jabref. I don't believe that jabref actually translates to lower-case, though I may be wrong.

Copy link
Member

Choose a reason for hiding this comment

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

yeah, jabref keeps the capitalization as is it seems. All the same to me. :-)

@tjhei tjhei merged commit c72b1f4 into geodynamics:main Aug 4, 2022
@bangerth bangerth deleted the citations branch August 4, 2022 22:16
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

3 participants