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
Disable background thread when networking is off #3234
Disable background thread when networking is off #3234
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
FYI, this timed out on pycicle so I'll take this as an opportunity to check what breaks with networking off (or what tests etc. should be disabled, if they're not already disabled). Consider this low-priority exploration. |
@msimberg Thanks! And yes, tests will be a problem - we never really made this consistent. |
The flag can still be set but has no effect.
…elevant options in hpxrun.py Force one locality for all tests with more than one locality and don't pass localities and parcelport options on from hpxrun.py to the test executable (since the options are disabled and cause the test to fail).
6ee8cad
to
9a62c15
Compare
d47a3ea
to
d712398
Compare
a43e95b
to
b2ebc9d
Compare
This ran successfully with I've
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot!
Proposed Changes
scheduling_loop
ifHPX_HAVE_NETWORKING=OFF
@biddisco Might give you a tiny speedup for Cholesky, unless you've already disabled the background thread.
WIP as I will still run the tests with pycicle and networking off.