Repository navigation
Keep baron a title, measured alone against its given names (#913) - #1054
Merged
Merged
Conversation
Given a given reading by itself, baron moved 129 local-corpus names: about 58 given names (FEC 31 of 31) against about 66 titles, mostly jrc-names' peers and 15 PSC filers who typed Baron in the title box. Record the decline on the word, in decisions/H9 (correcting the earlier spike's PSC count), and the lesson in nameparser/vocab/AGENTS.md. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Correct the tally (FEC 30 of 31, the surnames BARON GHORAB and Baron Crespo counted as neither: about 58 given, 63 titles, 8 neither), six Turkish or Turkmen PSC filers not nine, the recompute recipe's missing as_of, a positional pointer, and the AGENTS.md summary. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Records the decision to keep
barona title. No parser behavior changes:nameparser/vocab/en.tomlgains a note on baron's title reading and nothing else, so the compiled vocabulary is byte-identical.The measurement
baronwas given a datedgivenreading on its own and measured against mastera1b3e186(tools/differential/measure.py --base, 22 steps). It moved 129 local-corpus names, each from title 'Baron' to a name word.SMITH, BARON P), orcid-researchers 7, patentsview-inventors 4, ctgov-officials 2 (Baron Lonner, MD), the committed snapshot 1 (Baron Davis), psc-individuals 6, and about 8 in jrc-names (Baron Waqa).Baron Falconer of Thoroton), 15 in psc-individuals, 2 in tse-donors and 1 in ukec-individuals (Baron David De Rothschild).Baron Cohen,Baron Crespo(Enrique Barón Crespo) andBARON GHORAB, NEILDA.PSC's filers type the title into a box of its own, and 15 of the 21 PSC filers typed
Baronthere, six of them Turkish or Turkmen. That makes the change a net loss. It also undercuts rules/H9's reason for accepting a bare title read as a given name, which holds for US records but not for the UK's.What changes
nameparser/vocab/en.toml: the counts and the reason, in a note onbaron's title reading, recorded the same way asjudge's.docs/design/decisions/H9-title-phrase.md:Baron Smith of Kensingtonand a leadingThe.nameparser/vocab/AGENTS.md: a lesson to count a fielded corpus's moves by its filed boxes (local_agreement.py --list), not by reading the names.A leading
Thebefore a title that is a title only inside a phrase (The Marquis de Sade→ given 'The Marquis' on master) is being handled separately.🤖 Generated with Claude Code