Skip to content

[back] Fix journal article entries in bibliography#8165

Closed
morinmorin wants to merge 2 commits into
cplusplus:mainfrom
morinmorin:fix_journal_article_list
Closed

[back] Fix journal article entries in bibliography#8165
morinmorin wants to merge 2 commits into
cplusplus:mainfrom
morinmorin:fix_journal_article_list

Conversation

@morinmorin

Copy link
Copy Markdown
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".

Comment thread source/back.tex
\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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

tkoeppe commented Aug 18, 2025

Copy link
Copy Markdown
Contributor

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

@tkoeppe

tkoeppe commented Aug 18, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor Author

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

@tkoeppe

tkoeppe commented Aug 18, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor Author

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

@tkoeppe

tkoeppe commented Aug 18, 2025

Copy link
Copy Markdown
Contributor

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
@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Oct 31, 2025
@tkoeppe

tkoeppe commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

Editorial meeting decision: we generally like the style we use, even though it differs from the academic style. We might even entertain the idea of also italicizing the titles of the three online references.

@tkoeppe tkoeppe closed this Nov 5, 2025
@tkoeppe tkoeppe removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs rebase The pull request needs a git rebase to resolve merge conflicts.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants