-
-
Notifications
You must be signed in to change notification settings - Fork 427
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
Addressing remaining Stdexec issues #6534
Comments
This minimal example can be used to reproduce the 2nd issue. Alternatively, it can also be reproduced by running any of the following unit tests:
with one of these command line arguments:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Remaining issues
std
synchronization primitives, which clash with HPX's (e.g.stdexec::run_loop
,this_thread::sync_wait
usestd::mutex
andstd::condition_variable
. This causes some tests to deadlock. Thanks to @zhekemist for identifying this! The following commit fixed a bug caused by this interaction: d06c9ad.algorithm_transform_mpi
andmpi_ring_async_executor
have been disabled due to issue # 2 highlighted above. Other tests may be failing too, but it was noticed that there are MPI tests that fail but doreturn 0
so they are not marked as failing. This should be corrected too.Due to these issues, Stdexec won't be enabled by default. To activate there needs to be an explicit definition:
HPX_WITH_STDEXEC=ON
The text was updated successfully, but these errors were encountered: