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

Inclusive scan #716

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

Conversation

khlebnikov
Copy link
Contributor

Concept is still the same (based on the Standard) - there's no clear analogy for semigroup when the accumulator type is different from value type.

@ericniebler
Copy link
Owner

I haven't forgotten about this. Day job has kept me busy. I suspect there are more meaningful concepts to be found here, I just haven't had time to sit down with the algorithm to confirm.

@khlebnikov
Copy link
Contributor Author

I certainly understand. I'm sure there is a better concept hidden inside, but I wanted to push the changes before madness started (I moved countries and started a new job in the meanwhile :) )

@voivoid
Copy link

voivoid commented Dec 1, 2018

It would be great if this feature is merged

@ericniebler
Copy link
Owner

ericniebler commented Dec 4, 2019

Concept-ifying the numerics algorithms is the work of @cjdb in p1813, however I see that he didn't yet get around to inclusive_scan and exclusive_scan. I would like to see that work and this harmonized.

Also @cjdb, any interest in turning P1813 into a PR for range-v3? nudge, nudge

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.

None yet

3 participants