Skip to content

Add non-recursive reductions - #83

Merged
inducer merged 2 commits into
inducer:mainfrom
majosm:non-recursive-reductions
Sep 4, 2021
Merged

Add non-recursive reductions#83
inducer merged 2 commits into
inducer:mainfrom
majosm:non-recursive-reductions

Conversation

@majosm

Copy link
Copy Markdown
Collaborator

Used by #71. Depends on #81 and #82.

@majosm
majosmforce-pushed the non-recursive-reductions branch from 3e210a7 to 2d51f3eCompareAugust 27, 2021 20:52
@majosm
majosmforce-pushed the non-recursive-reductions branch from 2d51f3e to c4c6b51CompareAugust 30, 2021 20:36
@majosm
majosm marked this pull request as ready for review August 30, 2021 21:33
@majosm
majosm requested review from alexfikl and inducerAugust 30, 2021 21:33

@alexfiklalexfikl left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small nitpick in the docs, otherwise this looks good to me! 👍

Comment threadarraycontext/container/traversal.py Outdated
Co-authored-by: Alex Fikl <alexfikl@gmail.com>
@majosm
majosmforce-pushed the non-recursive-reductions branch from 9736add to 4a47e11CompareSeptember 1, 2021 14:48
@inducer
inducer merged commit 4b184f8 into inducer:mainSep 4, 2021
@inducer

Copy link
Copy Markdown
Owner

LGTM, thanks!

Sign up for freeto 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.

3 participants

@majosm@inducer@alexfikl