Skip to content
This repository has been archived by the owner on Feb 2, 2024. It is now read-only.

Migrating to Numba 0.51.2 #926

Merged

Conversation

kozlov-alexey
Copy link
Contributor

@kozlov-alexey kozlov-alexey commented Sep 23, 2020

Adds prefer_literal flag to sdc overload decorators with default value True to satisfy previous Numba behavior.

Following problems were solved during transition:

  • Added type-checks to functions decorated with intrinsic that require literal arguments (e.g. getitem accessors) to handle new Numba behavior to prefer_literal=False default typing (introduced in Add prefer_literal option to overload API numba/numba#6058)
  • Fixed incorrect type-checks in fillna and astype overloads to avoid returning wrong function implementation when compiling with non-literal args first.

NOTE: this requires patch for numba to add support of options
to intrinsic decorator to be applied first
@pep8speaks
Copy link

pep8speaks commented Sep 24, 2020

Hello @kozlov-alexey! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2020-09-27 05:11:09 UTC

@kozlov-alexey kozlov-alexey changed the title [WIP] Migrating to Numba 0.51.2 Migrating to Numba 0.51.2 Sep 25, 2020
Copy link
Contributor

@Hardcode84 Hardcode84 left a comment

Choose a reason for hiding this comment

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

Write in description why these changes were needed, LGTM otherwise

@kozlov-alexey
Copy link
Contributor Author

Write in description why these changes were needed, LGTM otherwise

Done.

@Hardcode84 Hardcode84 merged commit ee80555 into IntelPython:master Sep 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants