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 missing values for some provisional constants #485

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

raffenet
Copy link
Contributor

These were missed when we assigned values to most of the constants in the document.

These were missed when we assigned values to most of the constants in
the document.

Signed-off-by: Ken Raffenetti <[email protected]>
@raffenet
Copy link
Contributor Author

Marking this as errata. The ABI repo already has these values in the standard headers.

@raffenet
Copy link
Contributor Author

raffenet commented Jun 5, 2023

Please use emoji reactions ON THIS COMMENT to indicate your position on this proposal.

  • You do not need to vote on every proposal
  • If you have no opinion, don't vote - that is also useful data
  • If you've already commented on this issue, please still vote so
    we know your current thoughts
  • Not all proposals solve exactly the same problem, so we may end
    up accepting proposals that appear to have some overlap
    This is not a binding majority-rule vote, but it will be a very
    significant input into the corresponding ASC decision.

Here are the meanings for the emojis:

  • Hooray or Rocket: I support this so strongly that I
    want to be an advocate for it
  • Heart: I think this is an ideal solution
  • Thumbs up: I'd be happy with this solution
  • Confused: I'd rather we not do this, but I can tolerate it
  • Thumbs down: I'd be actively unhappy, and may even consider
    other technologies instead

If you want to explain in more detail, feel free to add another
comment, but please also vote on this comment.

@jjhursey jjhursey added the Eligible Eligible for consideration by ASC label Jul 3, 2023
@jjhursey
Copy link
Member

PMIx ASC 3Q 2023 Voting results:

  • Yes: 6
  • No: 0
  • Abstain: 0
  • Result: Pass as Errata. Can be merged at the discretion of Co-Chairs/RMs.

@jjhursey jjhursey added the First Vote Passed ASC first vote passed label Jul 20, 2023
@abouteiller abouteiller self-requested a review January 25, 2024 06:30
@abouteiller abouteiller merged commit 0a7d0f8 into pmix:master Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Eligible Eligible for consideration by ASC Errata First Vote Passed ASC first vote passed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants