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

Bibliography cleanups #2914

Merged
merged 19 commits into from Mar 13, 2021
Merged

Conversation

eggrobin
Copy link
Member

@eggrobin eggrobin commented Mar 13, 2021

This change:

  • moves the biblatex package and its customizations to the basestyle template;
  • moves bibliography printing at the end of the document to the basestyle template;
  • ensures that bibliography labels are consistent across all documents—[Fuk11a] was previously being cited as [Fuk11] in Fukushima.pdf;
  • customizes labelalpha so that it includes the custom userd (user-defined field, disambiguator) field;
  • ensures that no ambiguity is left once userd is factored in, i.e., that the automated disambiguator extraalpha is empty, ensuring stability of labels;
  • sets userd:
    • consistently with the existing labels if those were cited (most cases) changing the .bib ID to match if needed, or,
    • consistently with the .bib IDs otherwise ([Fan91a], [Fan91b]);
  • changes @incollection to @inproceeding where the collection is specifically conference proceedings;
  • changes @incollection to @article for [Del90], since that collection is a periodical with an ISSN;
  • adds eventdate and venue to all @inproceeding entries, as well as some other fields as appropriate.

In order to make diffing easier, this change does not re-alphabetize the entries; this will be done in a subsequent pull request using the usual script.

@@ -1,8 +1,5 @@
\documentclass[10pt, a4paper, twoside]{basestyle}

\usepackage[backend=biber,firstinits=true,maxnames=100,style=alphabetic,maxalphanames=4,doi=true,isbn=false,url=false,eprint=true]{biblatex}
\bibliography{bibliography}

Copy link
Member

Choose a reason for hiding this comment

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

This changes the spacing between the bibliographical id and the bibliographical entry in the References section (previously the entry started roughly below the "n" of "References", now it's below the "e". This less visually pleasant. In particular in this document this leads to a ridiculously overfull hbox when in the [Wes17] reference.

Copy link
Member Author

Choose a reason for hiding this comment

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

This was because of the nocite{*} in another refsegment (whose bibliography is not printed) in the same refsection (the margin shifted to accomodate the widest possible label).

This was a way to ensure globally consistent labels with automated disambiguation; now that we have userd we no longer need that.
Note that this means that global uniqueness of the references will be checked only when compiling bibliography (but we should probably make the CI do that anyway).

Done.

Copy link
Member

Choose a reason for hiding this comment

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

There are still differences in the way that the URLs get folded in [PL10] and [Wes17] (and the latter is still ugly).

Copy link
Member Author

@eggrobin eggrobin Mar 13, 2021

Choose a reason for hiding this comment

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

Ah, this is because eprint now starts a line per the global style, and our current settings forbid mid-word linebreaking in a URL. I will try to tame the url options.

Copy link
Member Author

Choose a reason for hiding this comment

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

Done, there’s a biblatex setting for that (found in https://tex.stackexchange.com/a/134281).

title = {Repeat Ground Track Lunar Orbits in the Full-Potential Plus Third-Body Problem},
volume = {3},
Copy link
Member

Choose a reason for hiding this comment

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

Why did we lose the volume?

Copy link
Member Author

Choose a reason for hiding this comment

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

As best I can tell it was nonsense.
Guidance, Navigation, and Control and Co-located Conferences is unnumbered, so it was not a number (a lot of volumes were « volume within the series » which should go into number in @inproceedings).
AIAA/AAS Astrodynamics Specialist Conference and Exhibit (2006) does not appear to be partitioned in volumes either: https://arc.aiaa.org/doi/book/10.2514/MAST06.

Note that the previous value of series, Collection of Technical Papers, seems nonsensical.

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

@eggrobin eggrobin Mar 13, 2021

Choose a reason for hiding this comment

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

Ah, it looks like in print these proceedings appeared in the three-volume ISBN 1563478226, A collection of technical papers:

  1. https://www.tib.eu/en/search/id/TIBKAT:521489660/A-collection-of-technical-papers-AIAA-AAS-Astrodynamics?cHash=6349bf79113ed67885ab83701e652639;
  2. https://www.tib.eu/en/search/id/TIBKAT:521489873/A-collection-of-technical-papers-AIAA-AAS-Astrodynamics?cHash=cd433c128bcf43589e005c9f781fdb37;
  3. https://www.tib.eu/en/search/id/TIBKAT:521489903/A-collection-of-technical-papers-AIAA-AAS-Astrodynamics?cHash=cb1d3358a90d8b8658ca3bb358e99946.

See https://www.tib.eu/en/search/id/BLCP:CN065746138/Repeat-Ground-Track-Lunar-Orbits-in-the-Full-Potential?cHash=f086b9f29f698e135b8d8d7aa257c363.

This print version has page numbers 1935–1954 in that citation, whereas the ARC PDF is numbered 1–20…

If we want to cite the print book we should do the same for the other paper from the AIAA/AAS Astrodynamics Specialist Conference, [Jon12], in https://www.tib.eu/en/search/id/TIBKAT:769080677/AIAA-AAS-Astrodynamics-Specialist-Conference-2012?cHash=190c9597dcb0a79c120396334a3fb587—note that for that one the POD publisher is Curran, and the AIAA would need to move from publisher to organization.

Copy link
Member Author

Choose a reason for hiding this comment

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

Added the print metadata to both entries.

@pleroy pleroy added the LGTM label Mar 13, 2021
@eggrobin eggrobin merged commit 6e184be into mockingbirdnest:master Mar 13, 2021
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