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
Adapted parallel::{count|count_if} for Ranges TS (see #1668) #3169
Conversation
@cogle could you please add the new files to the build system for the documentation as well (here: https://github.com/STEllAR-GROUP/hpx/blob/master/docs/CMakeLists.txt#L115) ? |
It appears I failed the following test Just wondering what may be the source of this failure may be. |
In general this report should be available under artifacts on circleci, but you have to be signed in to see it. |
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 a lot!
@biddisco what are some possible reasons for failing the |
@cogle Actually at the moment even we can't tell what went wrong in that one because the results are not viewable. Most likely the failed test is completely unrelated to your changes but once things are back to normal clicking on details should get you an idea of what's failed. |
Ok thanks! I was wondering if I was like with circleci missing a way to view the results. |
Extends the
count
andcount_if
algorithms for Ranges.Proposed Changes
count
,count_if