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

Make Zoltan's Hyperedge Size Threshold Runtime Controllable #6036

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

Conversation

bska
Copy link
Member

@bska bska commented Feb 25, 2025

This commit introduces a new hidden runtime parameter,

ZoltanPhgEdgeSizeThreshold (--zoltan-phg-edge-size-threshold, double, default value = 0.35)

which maps to Zoltan's low-level control parameter

PHG_EDGE_SIZE_THRESHOLD

that controls which hypergraph edges to omit/discard. We add a new parameter to setupZoltanParameters() and thread the command line parameter through as an argument to this function.

This is to enable runtime experimentation with this aspect of the partitioning algorithm, but end-users should typically not alter the default value without good reason.

@bska
Copy link
Member Author

bska commented Feb 25, 2025

As a secondary benefit, this PR will obviate the PHG_EDGE_SIZE_THRESHOLD part of OPM/opm-grid#813.

@bska
Copy link
Member Author

bska commented Feb 25, 2025

jenkins build this please

@bska bska force-pushed the expose-zoltan-phg-edge-size-param branch from 48457fe to 93ead52 Compare February 26, 2025 11:08
@bska
Copy link
Member Author

bska commented Feb 26, 2025

jenkins build this please

@bska bska force-pushed the expose-zoltan-phg-edge-size-param branch 2 times, most recently from 9908c4c to 8915efc Compare February 26, 2025 17:23
@bska
Copy link
Member Author

bska commented Feb 26, 2025

jenkins build this please

@bska bska force-pushed the expose-zoltan-phg-edge-size-param branch from 8915efc to a34d2ef Compare February 27, 2025 13:01
@bska
Copy link
Member Author

bska commented Feb 27, 2025

jenkins build this please

@bska bska force-pushed the expose-zoltan-phg-edge-size-param branch from a34d2ef to fd151f0 Compare March 3, 2025 16:35
@bska
Copy link
Member Author

bska commented Mar 3, 2025

jenkins build this please

@bska bska force-pushed the expose-zoltan-phg-edge-size-param branch from fd151f0 to c9911fa Compare March 4, 2025 18:03
@bska
Copy link
Member Author

bska commented Mar 4, 2025

jenkins build this please

@bska bska force-pushed the expose-zoltan-phg-edge-size-param branch 2 times, most recently from dc15154 to a7936dc Compare March 5, 2025 13:39
bska added 2 commits March 5, 2025 16:47
Slight rewording and splitting long lines for the most part.
This commit introduces a new hidden runtime parameter,

    ZoltanPhgEdgeSizeThreshold (--zoltan-phg-edge-size-threshold,
    double, default value = 0.35)

which maps to Zoltan's low-level control parameter

    PHG_EDGE_SIZE_THRESHOLD

that controls which hypergraph edges to omit/discard.  We add a new
parameter to 'setupZoltanParameters()' and thread the command line
parameter through as an argument to this function.

This is to enable runtime experimentation with this aspect of the
partitioning algorithm, but end-users should typically not alter the
default value without good reason.
@bska bska force-pushed the expose-zoltan-phg-edge-size-param branch from a7936dc to 9b935ad Compare March 5, 2025 15:47
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