Skip to content

Add some node-load-balancing test coverage#4404

Open
roystgnr wants to merge 1 commit intolibMesh:develfrom
roystgnr:load_balanced_nodes_coverage
Open

Add some node-load-balancing test coverage#4404
roystgnr wants to merge 1 commit intolibMesh:develfrom
roystgnr:load_balanced_nodes_coverage

Conversation

@roystgnr
Copy link
Member

I never got around to upgrading these tests to "rigorous testing of partitioner quality" rather than "testing to see if those code paths outright crash or corrupt partitioning", so this commit languished, but even a little test coverage here is better than nothing.

@roystgnr
Copy link
Member Author

Huh, maybe that's why I never opened up a PR for this - we need either a behavior change or more complicated scripting for the petscpartitioner option not to break make check with non-PETSc builds.

But on the other hand, that --load-balanced-nodes-linear failure on 4 and 5 processors looks real, so we really do need some test coverage of all these.

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.

1 participant