Skip to content

[back] Fix journal article entries in bibliography #8165

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

morinmorin
Copy link
Contributor

This PR applies the following corrections and formatting unifications.

  • Fixes author name typo: "R. Client Whaley" should be "R. Clint Whaley". (c.f. https://doi.org/10.1145/567806.567807 )
  • Adds missing page numbers.
  • Unifies article title format: Roman style with sentence capitalization.
  • Unifies journal title format: italicized.
  • Unifies journal volume/issue format: "vol. X, no. Y".

\doccite{Fast linear algebra is stable},
Numerische Mathematik 108 (59--91), 2007.
Fast linear algebra is stable,
\doccite{Numerische Mathematik}, Vol.\ 108, No.\ 1, pp.\ 59--91, 2007.
Copy link
Member

Choose a reason for hiding this comment

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

We seem to \doccite the title of the book / article elsewhere in this list, not (necessarily) the journal name.

Agreed with the rest of the adjustments.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, will fix.
Just to be sure: is it okay to continue with the current "articles title in italic" style? In usual bibliographic styles, article titles are written in roman and journal names are written in italic. Examples (C.7.2) in ISO 690:2010 follow this convention as well.

Copy link
Member

Choose a reason for hiding this comment

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

@tkoeppe, what do you want here?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, I was aware that we're not following the usual conventions here. (E.g. bibtex would do it right, but when we started we just had a very small set of items.) This is a nice improvement.

@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 18, 2025

Please update the commit message to be a bit more specific.

@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 18, 2025

Actually, please split this up into the actual corrections (typos and missing page numbers), and separately the style changes.

This change applies the following corrections.
- Corrects misspelling of name to R. Clint Whaley.
- Adds missing page numbers.
This change applies the following unifications.
- Unifies article title format: roman style with sentence capitalization.
- Unifies journal title format: italicized.
- Unifies journal volume/issue format: "vol. X, no. Y".
@morinmorin morinmorin force-pushed the fix_journal_article_list branch from f1fc421 to 2a837a4 Compare August 18, 2025 11:19
@morinmorin
Copy link
Contributor Author

I split this into two commits. Please let me know if you'd like me to split this into two PRs.

@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 18, 2025

Thanks!

I suppose what's somewhat unfortunate about the established style is that there's no recognizable distinction between the author list and the title. This has always irked me, and that's why I was fine with our existing style, even though it goes against some convention...

@morinmorin
Copy link
Contributor Author

Thanks for sharing your perspective! I can totally see how that lack of distinction could be annoying.

@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 18, 2025

I've applied the fixes for now, and we can maybe leave this one open for discussion for a while.

@tkoeppe tkoeppe added the decision-required A decision of the editorial group (or the Project Editor) is required. label Aug 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
decision-required A decision of the editorial group (or the Project Editor) is required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants