Skip to content

ParallelAccelerator no long will convert StaticSetItem to SetItem because record arrays require StaticSetItems. - #6801

Merged
sklam merged 6 commits into
numba:masterfrom
IntelLabs:issue6704
Mar 23, 2021
Merged

sklam merged 6 commits into
numba:masterfrom
IntelLabs:issue6704

Conversation

@DrTodd13

@DrTodd13 DrTodd13 commented Mar 5, 2021

Copy link
Copy Markdown
Contributor

Resolves #6704.

@sklam sklam left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I've suggested additional test to improve coverage.

Comment thread numba/tests/test_parfors.py
Comment thread numba/tests/test_parfors.py Outdated
@sklam sklam added 4 - Waiting on author Waiting for author to respond to review and removed 3 - Ready for Review labels Mar 8, 2021
DrTodd13 and others added 2 commits March 8, 2021 15:48
Co-authored-by: Siu Kwan Lam <1929845+sklam@users.noreply.github.com>
@DrTodd13 DrTodd13 added 4 - Waiting on reviewer Waiting for reviewer to respond to author and removed 4 - Waiting on author Waiting for author to respond to review labels Mar 8, 2021
@stuartarchibald

Copy link
Copy Markdown
Contributor

Open/Close was to try and re-start the RTD docs build.

@stuartarchibald stuartarchibald left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the patch, couple of minor things else looks good.

Comment thread numba/tests/test_parfors.py Outdated
Comment thread numba/parfors/parfor.py
@stuartarchibald stuartarchibald added 4 - Waiting on author Waiting for author to respond to review Effort - short Short size effort needed and removed 4 - Waiting on reviewer Waiting for reviewer to respond to author labels Mar 22, 2021
DrTodd13 and others added 2 commits March 22, 2021 11:00
Co-authored-by: stuartarchibald <stuartarchibald@users.noreply.github.com>
Co-authored-by: stuartarchibald <stuartarchibald@users.noreply.github.com>
@DrTodd13 DrTodd13 added 4 - Waiting on reviewer Waiting for reviewer to respond to author and removed 4 - Waiting on author Waiting for author to respond to review labels Mar 22, 2021

@stuartarchibald stuartarchibald left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the patch and fixes. @sklam's unit test is added so I think everything is resolved.

@stuartarchibald stuartarchibald added 5 - Ready to merge Review and testing done, is ready to merge and removed 4 - Waiting on reviewer Waiting for reviewer to respond to author labels Mar 23, 2021
@sklam
sklam merged commit fe31b54 into numba:master Mar 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 - Ready to merge Review and testing done, is ready to merge Effort - short Short size effort needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot lower setitem on a record in a parallel loop

4 participants