Skip to content

Update mailmap to latest commit#31815

Merged
tacaswell merged 4 commits into
matplotlib:mainfrom
QuLogic:update-mailmap
Jun 4, 2026
Merged

Update mailmap to latest commit#31815
tacaswell merged 4 commits into
matplotlib:mainfrom
QuLogic:update-mailmap

Conversation

@QuLogic
Copy link
Copy Markdown
Member

@QuLogic QuLogic commented Jun 4, 2026

PR summary

This was last updated in cee7684, which added new contributors up to 0b61e32. This PR handles everything up to 99ae86b.

While going through the GitHub stats pages, I found out some annoyances with how GitHub does things. Specifically, anything a contributor "does" online uses the information from the public profile. I put "does" in quotes, because while it does cover explicit actions like committing in the web browser, it also includes one action not initiated directly by a user, but instead by a maintainer: squash merging a PR. In addition, when the script asks GitHub for the authors of a PR, it returns all the authors of the original commits. This means for every PR that is squash merged, we have the chance to count it as (at least) two authors instead of one.

This means that our author count on the GitHub stats pages can sometimes be highly inflated. I have updated the script so that it works better post-release, and regenerated those pages for releases after 0b61e32. Note that in the extreme case, some of the v3.10.x pages were generated with --since v3.10.0 instead of the previous release, so there are quite a few extra not even due to duplicate authors.

All changes and reasons why

I am putting accounts in a code block so to not ping ~75 people on one PR:

  • @34j: Initial commit in Simplify colored_line() implementation in Multicolored lines example #29954 with proton email, followups committed on GitHub, using masked email. Canonicalized to proton email.
  • @AMAN194701: commits locally had Title Case name, on GitHub used UPPER CASE.
  • @AasmaGupta: Some invalid email from local config. Canonicalized to gmail.
  • @AdamOrmondroyd: commits locally were gmail, on GitHub used the masked email.
  • @AndrGutierrez: there were two emails and two names (but I think one was a misspelling). Canonicalized on the one that was known to GitHub last (i.e., used for the squash merge.)
  • @AnnaMastori: Just a minor spacing error, went with the name as in profile.
  • @Atharva2012: this was a typo: DOC: Add axis sharing section to subplot_mosaic guide #31720 (comment)
  • @Cemonix: all the name, email, and masked email came in via DOC: correctly specify return type of figaspect #29545.
  • @Chirag3841: the squash merge appears to have used the masked email, but all the commits used gmail.
  • @CouldNot: commits locally were gmail, on GitHub used the masked email.
  • @DerWeh: the latter email is the second commit of DOC: correctly specify return type of figaspect #29545 (along with a full name), but it is not linked to the GitHub account, so I assume the former is preferred now (and recent activity has not used the full name either, nor does the profile show one, so I won't add it here.)
  • @Dusch4593: Two names for same email. Canonicalized on GitHub profile name.
  • @GameRoMan: Used the name as given in the profile.
  • @Impaler343: commits locally were gmail, on GitHub used the masked email.
  • @Jacob-Stevens-Haas: Two names for same email. Canonicalized on GitHub profile name.
  • @Kaustbh: commits locally were gmail, on GitHub used the masked email.
  • @Khushikela29: commits locally were gmail, on GitHub used the masked email.
  • @Logan-Pageler: commits locally were gmail, on GitHub used the masked email.
  • @OdileVidrine: commits locally were gatech, on GitHub used the masked email.
  • @ZPyrolink: commits locally were gmail, on GitHub used the masked email.
  • @aditya-singh597: One email used locally, but a different one for the profile. Canonicalized on the profile one?
  • @agautam478: commits locally were uber, on GitHub used the masked email.
  • @albus-droid: commits had a mix of name/username for name. Canonicalized on GitHub name in profile.
  • @alphanoobie: commits locally were gmail, on GitHub used the masked email.
  • @amishamehta99: DOC: make nightly download command one line so it works on Windows #31231 and DOC: Fix rendering of quiver documentation #31252 are Name ; not sure where the masked email came from, but it is linked to the email on GitHub.
  • @ananya314: commits locally were gmail, on GitHub used the masked email.
  • @andrew-fennell: two emails, but both are linked to GitHub. Canonicalized on the one used in all commits but one.
  • @aseriesof-tubes: The email is invalid, and I can't find any other commits here, so I used the GitHub masked email for the user from the PR that introduced it.
  • @beelauuu: commits locally were .edu, on GitHub used the masked email.
  • @buddy0452004: Used the name as given in the profile.
  • @chahak13: both emails are linked to GitHub, but went with the one used more often.
  • @chaoyihu: commits locally were gmail, on GitHub used the masked email.
  • @ckcherry23: commits locally were gmail, on GitHub used the masked email.
  • @devRD: Two names for same email. Canonicalized on GitHub profile name.
  • @dkweiss31: Original commit in Update secondary_axis tutorial #29073 uses the .edu email and all subsequent commits were using GitHub suggestions, so used the masked email.
  • @dmatos2012: commits locally used two emails used, on GitHub used the masked email. Canonicalized on the most used email.
  • @esibinga: commits locally were gmail, on GitHub used the masked email.
  • @greglucas: Canonicalized on the more recent email.
  • @hasanrashid: commits locally were gmail, on GitHub used the masked email.
  • @impact27: commits locally were ac.uk, on GitHub used the masked email.
  • @jayaprajapatii: Just a minor spacing error, went with name as in profile.
  • @joddeepesh-cloud: Two names for same email. Canonicalized on the GitHub profile name.
  • @juanis2112: Two names for same email. Canonicalized on GitHub profile name.
  • @konmenel: commits locally were gmail (with a mix of full name or username as name), on GitHub used the masked email.
  • @kyracho: both emails are linked to the GitHub account, but the one I used here is the one in the profile.
  • @lukashergt: both emails are linked to the GitHub account, but the one I used here is the one in the profile.
  • @mathause: commits locally were ethz, on GitHub used the masked email.
  • @meeseeksmachine: Renamed, so set to new name.
  • @melissawm: A few mixed names/emails, so canonicalize on the current one.
  • @mhvk: Went with the name in the latest commit, though the GitHub profile doesn't have a middle name at all, so unsure about this one.
  • @miriamsimone: Multiple names for the single email; went with the full name.
  • @mkrake: commits locally were gmail, on GitHub used the masked email; also the username changed.
  • @n-takumasa: Three names for same email. Canonicalized on latest name.
  • @nrnavaneet: commits locally were gmail, on GitHub (either suggestions or squash merges) used the masked email.
  • @pedrom2002: Canonicalized on name from profile.
  • @pedrompecanha: commits locally were gmail, on GitHub used the masked email.
  • @photoniker: commits locally were gmail, on GitHub used the masked email, I think.
  • @pirzada-ahmadfaraz: commits locally were gmail, on GitHub used the masked email.
  • @r3kste: Appears sometimes as Anton, sometimes as anTon, sometimes as r3kste, but the name on the profile is Anton now, so go with that.
  • @raphaelquast: Two names for same email. Canonicalized on GitHub profile name.
  • @rcomer: Probably masked email came in from changes made directly on GitHub; went with the Met email.
  • @rwpenney: commits locally were personal domain, on GitHub used the masked email.
  • @saranti: name appears 4 different ways with the same email, went with the name in the profile.
  • @saumyacoder1709: the email address is invalid, and I can't find any other commits with a valid one, so use the masked GitHub email from FIX: Prevent warning when clearing axes with shared non-linear scale #30843.
  • @scottshambaugh: Not sure why there are two masked emails, and neither appears to be more recent, so I went with the more frequent one (125 vs 69 commits.)
  • @shriyakalakata: commits locally were gmail, on GitHub used the masked email; there was also a couple invalid emails that I canonicalized to this account.
  • @star1327p: full name is mostly used; not sure why the username snuck in a few places.
  • @stevezhang1999: first commit had an invalid email, and the rest were a mix of gmail and masked email. Canonicalized on the gmail.
  • @stiglers-eponym: commits locally were gmail, on GitHub used the masked email.
  • @tfpf: Replace tfpf with full name, as in profile. The username appears in older commits, but full name added later. There's also a gmail, but that seems to not be used any more.
  • @thrien: the email on the commit from Fix get_path for 3d artists #27440 was invalid, so I took the name/email from the GitHub profile.
  • @timhoffm: I've never seen the non-masked email before, and though they're in PRs you've made, the commits are no longer linked to your account. So I think those few commits were a mistake or an old email.
  • @tinezivic: all commits use gmail, except the one squash merged PR, which used the masked email, so go with the first one.
  • @tybeller: commits locally were gmail, on GitHub used the masked email.
  • @vittoboa: commits locally were gmail, on GitHub used the masked email.

AI Disclosure

None

PR checklist

QuLogic added 3 commits June 3, 2026 17:01
This was last updated in cee7684, which
added new contributors up to 0b61e32.
This commit handles everything up to 99ae86b.
a) fetch the end date from when the milestone was closed.
b) limit commits to end at the corresponding tag.
c) don't add extra to commit count when we're starting from an explicit
   tag.
@QuLogic QuLogic added this to the v3.11.0 milestone Jun 4, 2026
Comment thread tools/github_stats.py Outdated
Co-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>
@tacaswell tacaswell merged commit e58558a into matplotlib:main Jun 4, 2026
27 of 32 checks passed
@QuLogic QuLogic deleted the update-mailmap branch June 4, 2026 20:32
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