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

Reformat profiling examples #3

Merged

Conversation

nspark
Copy link

@nspark nspark commented Mar 11, 2020

This PR builds on openshmem-org#237, but is significant enough to not be strictly editorial. It updates the example listings to use the new macros, but slightly adjusts the caption text to suit the new structure.

The Profiling Interfaces Section Committee (@jlinford, @agrippa, @wrrobin, @swpoole) previously approved this on openshmem-org#343.

@wrrobin wrrobin changed the base branch from master to sec/profiling-interface March 11, 2020 20:53
@wrrobin
Copy link
Owner

wrrobin commented Mar 11, 2020

Thanks @nspark.

@jlinford, @agrippa, @swpoole Can we get at least one more review to get this merged? This PR has already been approved previously at openshmem-org#343.

@agrippa
Copy link
Collaborator

agrippa commented Mar 20, 2020

@wrrobin looks good to me.

@wrrobin wrrobin requested a review from agrippa March 20, 2020 19:54
@wrrobin
Copy link
Owner

wrrobin commented Mar 20, 2020

@agrippa Thanks for the approval.

@wrrobin wrrobin merged commit 14a0a12 into wrrobin:sec/profiling-interface Mar 20, 2020
wrrobin pushed a commit that referenced this pull request May 5, 2020
section/rma: replace default team with world team
wrrobin pushed a commit that referenced this pull request May 5, 2020
Deprecate short-typed wait/test & update tables to std AMO
wrrobin pushed a commit that referenced this pull request Oct 3, 2024
[Annex C] Remove trapped and replace with detect
wrrobin pushed a commit that referenced this pull request Oct 3, 2024
Updated example 14 to remove `shmem_long_sum_to_all`
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