Skip to content
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

Add authors for the v5.0 release #437

Merged
merged 13 commits into from
Apr 3, 2023

Conversation

abouteiller
Copy link
Contributor

@abouteiller abouteiller commented Jan 27, 2023

Create the 'acknowledgment' for contributors to the v5.0 release.

Since we are for the first time using the ASC procedures to create the document, the section organization reflects the roles accordingly.

At the moment the PR contains only people obtained from skimming the ASC governance 'roles'.

  • use commit history to complete the contributors list
  • use review history to find more contributors
  • use ASC Qx meeting notes to find more contributors
  • contact WG heads to check that major participants in WG discussions have been properly credited.
  • review list of institutions (in particular w.r.t. institutions that do not currently have a representative, but did during the v5.0 cycle)

Signed-off-by: Aurélien Bouteiller <[email protected]>
@abouteiller abouteiller added RFC Request for Comment Editorial labels Jan 27, 2023
@abouteiller abouteiller self-assigned this Jan 27, 2023
@jjhursey jjhursey requested review from dsolt and raffenet January 31, 2023 19:39
@jjhursey jjhursey added this to the PMIx v5 Standard milestone Jan 31, 2023
@jjhursey
Copy link
Member

Ken and Dave - Can you help work on this list? Maybe we can post a message to the mailing list for folks to review it by a particular deadline? The community can then add/remove names and correct affiliations.

@jjhursey
Copy link
Member

jjhursey commented Feb 7, 2023

FYI: I put this on the agenda for our monthly call.

Comment on lines 55 to 72
\item Ralph H. Castain
\item Joshua Hursey and David Solt
\item Dirk Schubert
\item John DelSignore
\item Aurelien Bouteiller
\item Michael A Raymond
\item Howard Pritchard
\item Brice Goglin
\item Kathryn Mohror
\item Thomas Naughton and Swaroop Pophale
\item William E. Allcock and Paul Rich
\item Michael Karo
\item Artem Polyakov
\item Ken Raffenetti and Shane Snyder
\item Norbert Eicker
\item Brice Goglin and Guillaume Mercier
\item Hari Subramoni and Nat Shineman
\item Martin Schulz
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm wondering if we should just alphabetize this list. Maybe we can add some notation to indicate which org or orgs and individual was associated with during the standardization timeframe.

Copy link
Contributor

Choose a reason for hiding this comment

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

New commit authors are Brian Barrett and Stephen Herbein. To the best of my knowledge there were no new PR reviewers. I used the gh tool to list all PRs merged into master since 2020-09-30, which is when v4 was branched off.

raffenet@glitch:pmix-standard/ ([email protected]*) $ gh pr list --search "is:pr is:closed merged:>2020-09-30 base:master" --limit 2000 --json reviews > v5-reviews.json
raffenet@glitch:pmix-standard/ ([email protected]*) $ cat v5-reviews.json| jq | grep login | sort | uniq
          "login": "SteVwonder"
          "login": "abouteiller"
          "login": "dsolt"
          "login": "jjhursey"
          "login": "kathrynmohror"
          "login": "naughtont3"
          "login": "raffenet"
          "login": "rhc54"
          "login": "schulzm"
          "login": "shanedsnyder"

@raffenet
Copy link
Contributor

Just added two more commits after @naughtont3 sent over the data he scraped from the meeting notes document. The last commit isolates the one-time attendees in case we can to do any filtering there.

raffenet and others added 3 commits March 9, 2023 11:24
Co-authored-by: Dan Holmes <[email protected]>
Co-authored-by: Thomas Naughton <[email protected]>
Thanks to Michael Karo for pointing out.
@raffenet raffenet marked this pull request as ready for review March 9, 2023 17:35
@SteVwonder
Copy link
Contributor

SteVwonder commented Mar 10, 2023

I skimmed the Use Cases WG meeting notes (https://docs.google.com/document/d/1CfZJFvW5xzRhnxdFcCR7iKBsT7ag070NKgNO3_S1Ab8/edit). The main recurring names that I saw were @spophale, @jjhursey, and @rhc54. There was also a meeting with @hppritcha and @shanedsnyder to discuss their WGs crossovers. All of those names are already included in the acknowledgements. So I think we are all good from that WGs perspective.

PS - Congrats on the V5 release!

raffenet and others added 2 commits March 10, 2023 09:26
I confirmed over email to include Stephan and his supporting company
ParTec AG.
@shanedsnyder
Copy link
Contributor

For the storage working group, I just added the following contributors:

  • John Carrier (Intel)
  • Andreas Dilger (Whamcloud)
  • Bogdan Nicolae (ANL)

Signed-off-by: Aurélien Bouteiller <[email protected]>
@abouteiller
Copy link
Contributor Author

Looks ready to merge to me. @jjhursey ?

@raffenet
Copy link
Contributor

Looks ready to merge to me. @jjhursey ?

I don't have merge access but I agree this is good to go. I am off next week, but can work on creating the final document when I am back.

@jjhursey jjhursey merged commit 4fd2366 into pmix:master Apr 3, 2023
@jjhursey
Copy link
Member

jjhursey commented Apr 3, 2023

Merged. Thanks for all the work in tracking this down!

@abouteiller abouteiller linked an issue Apr 11, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Editorial RFC Request for Comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Contributors to v5.0 standard document
8 participants