Skip to content

Conversation

@nglaeser
Copy link

@nglaeser nglaeser commented Jul 22, 2024

The CryptoBib manual states that the author label rule is

Papers with four or more authors ==> First letter of each author's last name (up to 6)

(italics mine). This PR fixes 3 entries I found for 2 papers total whose labels didn't follow this rule:

  • PoPETS:MSHLHSHHMNC18 -> PoPETS:MSHLHS18
  • SP:CHIKMRsVW21 -> SP:CHIKMR21
  • EPRINT:CHIKMRsVW20 -> EPRINT:CHIKMR20

@nglaeser
Copy link
Author

@michelabdalla This PR has been open for a while, is there an issue I need to fix in order for it to be merged?

@michelabdalla
Copy link
Member

Hi @nglaeser, there are currently some other inconsistencies related to the labeling rule for papers with more than 6 authors and we are trying to figure out the best way to address it without causing problems.

FYI, @fabrice102 ran a test in late July and found out the following:
22 papers with >6 authors have <6 initials in key
821 papers with >6 authors have =6 initials in key
1389 papers with >6 authors have >6 initials in key

ludopulles added a commit to ludopulles/crypto_db that referenced this pull request Oct 1, 2024
Fixes more than what is discussed in pull request cryptobib#252:
- If a von-prefix occurs, add all the first letters to the shortened name, e.g.
	"Jeroen van de Graaf", Crypto 2010 => C:vdGraaf10
- Some names have been updated, but the labels were still outdated, e.g.
	[EC:Burmester90a] => [EC:BurDes90]
- Delicate care is taken into account that some keys were colliding so a "dis"-disambiguation is added, e.g.
	[EPRINT:UnrDom12] & [EPRINT:Unruh12] => [EPRINT:Unruh12a] & [EPRINT:Unruh12b]
@ludopulles
Copy link
Contributor

I found some more quirks apart from what that 6 author format script gave, like:

  • [EC:Burmester90a] should be [EC:BurDes90],
  • [EPRINT:UnrDom12] should be [EPRINT:Unr12], giving two entries the same key, so there should actually be [EPRINT:Unr12a] and [EPRINT:Unr12b].

See https://github.com/ludopulles/cryptobib_tools/blob/master/format_entry.py for the script to fix those, it might need the modified lib in https://github.com/ludopulles/cryptobib_lib to succesfully output the bibtex file.

My plan is to make a pull request for https://github.com/ludopulles/crypto_db/tree/fix-252 to address the issue raised here, when/if #260 is merged.

@nglaeser
Copy link
Author

nglaeser commented Oct 7, 2024

@michelabdalla I see, thank you for the response!

ludopulles added a commit to ludopulles/crypto_db that referenced this pull request Oct 18, 2024
Fixes more than what is discussed in pull request cryptobib#252:
- If a von-prefix occurs, add all the first letters to the shortened name, e.g.
	"Jeroen van de Graaf", Crypto 2010 => C:vdGraaf10
- Some names have been updated, but the labels were still outdated, e.g.
	[EC:Burmester90a] => [EC:BurDes90]
- Delicate care is taken into account that some keys were colliding so a "dis"-disambiguation is added, e.g.
	[EPRINT:UnrDom12] & [EPRINT:Unruh12] => [EPRINT:Unruh12a] & [EPRINT:Unruh12b]
@ludopulles ludopulles mentioned this pull request Oct 18, 2024
michelabdalla added a commit that referenced this pull request Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants