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

[ENH] Selectively enable MG C++ testing outputs for non-root processes #1616

Closed
seunghwak opened this issue May 24, 2021 · 2 comments
Closed
Assignees
Labels
improvement Improvement / enhancement to an existing function
Milestone

Comments

@seunghwak
Copy link
Contributor

Describe the solution you'd like
We previously disabled all the outputs from non-root processes in MG C++ testing and this will be undone in #1615 as this disables useful error messages as well.

We need a more sophisticated mechanism to disable only redundant test progress messages in long run.

We can investigate https://github.com/LLNL/gtest-mpi-listener or write our custom listeners for this purpose.

@seunghwak seunghwak added the ? - Needs Triage Need team to review and classify label May 24, 2021
@BradReesWork BradReesWork added improvement Improvement / enhancement to an existing function and removed ? - Needs Triage Need team to review and classify labels Aug 5, 2021
@BradReesWork BradReesWork added this to the 21.10 milestone Aug 5, 2021
@ChuckHastings ChuckHastings modified the milestones: 21.10, 21.12 Sep 22, 2021
@seunghwak
Copy link
Contributor Author

We may just close this issue as we can use

mpirun --output-filename <output-file-directory> ...

and outputs for each process will be stored in a separate file.

@ChuckHastings
Copy link
Collaborator

Closed. Workaround in @seunghwak comment above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement / enhancement to an existing function
Projects
None yet
Development

No branches or pull requests

3 participants