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

Current WIP on Hydra #5

Open
wants to merge 18 commits into
base: wip/pmi
Choose a base branch
from

Conversation

maksim-banin
Copy link

@maksim-banin maksim-banin commented Jun 5, 2017

Enabling spawns by doing most of the work in mpiexec

@raffenet
Copy link

raffenet commented Aug 8, 2017

This PR has a bunch of print statements that are visible during a default launch. Can you remove them? Both printf and HYD_PRINT.

@maksim-banin
Copy link
Author

Ok, I was planning to finish the name publishing functionality, but I can clean it up in the mean time. Thanks for taking your time and looking into this pull request.

@raffenet
Copy link

raffenet commented Aug 8, 2017

@maksim-banin ok, thanks. Just give us a heads-up when this is ready for review again.

Banin Maksim added 2 commits August 9, 2017 13:10
Since MPICH_TIMEOUT is not working, they would hang the whole testing process.
@raffenet
Copy link

@maksim-banin so what should be working after this series? It looks like spawn support is added. Your patches modify some of the testlist files to disable some tests, but I don't think those are related to these patches. E.g. namepub tests are currently broken, right?

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.

2 participants