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

remedy segfault with openmpi/intelmpi in MPI_Reduc #115

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

airmler
Copy link

@airmler airmler commented Nov 13, 2020

No description provided.

@solomonik
Copy link
Collaborator

I am a bit confused about the scope of changes in this request. Seems like mostly stuff for dry-runs, but also changes to model parameters are included (possibly due to more recent CTF commits?). Some tweaks to the time estimates seem like they would affect performance nontrivially, on the other hand if they are corrections/improvements, this might be valuable.

Sorry for not getting to this for such a long time.

@airmler
Copy link
Author

airmler commented Jul 29, 2021

Sorry I messed something up with my pull request. Please dont accept it
Edit: The idea was only to pull the commit c5c7cec, all other changes are experimental.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants