Skip to content

Migrate linear algebra functions from glue_lowering#8256

Merged
sklam merged 3 commits into
numba:mainfrom
apmasell:unglue_linalg
Aug 22, 2022
Merged

Migrate linear algebra functions from glue_lowering#8256
sklam merged 3 commits into
numba:mainfrom
apmasell:unglue_linalg

Conversation

@apmasell

Copy link
Copy Markdown
Contributor

Change functions in linalg module using glue_lowering to new overload
annotation.

Comment thread numba/np/linalg.py Outdated
Comment thread numba/np/linalg.py Outdated
Comment thread numba/np/linalg.py Outdated
Comment thread numba/np/linalg.py
Change functions in `linalg` module using `glue_lowering` to new `overload`
annotation.
@stuartarchibald stuartarchibald added this to the Numba 0.57 RC milestone Jul 26, 2022
@sklam sklam self-assigned this Jul 26, 2022

@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.

one nitpick.

Comment thread numba/np/linalg.py Outdated
@sklam sklam added 4 - Waiting on author Waiting for author to respond to review and removed 3 - Ready for Review labels Aug 1, 2022
Replace assertions for unreachable code into exceptions
Add performance warnings about non-contiguous arrays
@apmasell apmasell added 3 - Ready for Review and removed 4 - Waiting on author Waiting for author to respond to review labels Aug 3, 2022

@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.

Thanks for the patch!

@sklam sklam added 5 - Ready to merge Review and testing done, is ready to merge and removed 3 - Ready for Review labels Aug 22, 2022
@sklam sklam merged commit 2510610 into numba:main Aug 22, 2022
@apmasell apmasell deleted the unglue_linalg branch August 26, 2022 18:29
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants