Skip to content

Testhound/native cast 8138 - #8154

Merged
esc merged 2 commits into
numba:mainfrom
testhound:testhound/native-cast-8138
Jul 4, 2022
Merged

esc merged 2 commits into
numba:mainfrom
testhound:testhound/native-cast-8138

Conversation

@testhound

Copy link
Copy Markdown
Contributor

This pull request fixes pythons casts on float16 types in kernels as outline in this issue:

#8138

@testhound
testhound requested a review from gmarkall as a code owner June 13, 2022 13:35
@testhound

Copy link
Copy Markdown
Contributor Author

gpuci run tests

@stuartarchibald stuartarchibald added 2 - In Progress CUDA CUDA related issue/PR labels Jun 13, 2022
@gmarkall gmarkall added this to the Numba 0.57 RC milestone Jun 28, 2022
@gmarkall gmarkall self-assigned this Jun 28, 2022

@gmarkall gmarkall 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 fix - this looks good to me.

@esc / @stuartarchibald Could this have a CUDA buildfarm run please?

@gmarkall gmarkall added Pending BuildFarm For PRs that have been reviewed but pending a push through our buildfarm 4 - Waiting on CI Review etc done, waiting for CI to finish and removed 3 - Ready for Review labels Jul 1, 2022
@esc

esc commented Jul 4, 2022

Copy link
Copy Markdown
Member

Thanks for the fix - this looks good to me.

@esc / @stuartarchibald Could this have a CUDA buildfarm run please?

Build numba_smoketest_cuda_yaml_140 has started

@esc

esc commented Jul 4, 2022

Copy link
Copy Markdown
Member

Thanks for the fix - this looks good to me.
@esc / @stuartarchibald Could this have a CUDA buildfarm run please?

Build numba_smoketest_cuda_yaml_140 has started

Was all green!

@esc esc added BuildFarm Passed For PRs that have been through the buildfarm and passed and removed Pending BuildFarm For PRs that have been reviewed but pending a push through our buildfarm labels Jul 4, 2022
@gmarkall gmarkall added 5 - Ready to merge Review and testing done, is ready to merge and removed 4 - Waiting on CI Review etc done, waiting for CI to finish labels Jul 4, 2022
@esc
esc merged commit 15e4878 into numba:main Jul 4, 2022
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 BuildFarm Passed For PRs that have been through the buildfarm and passed CUDA CUDA related issue/PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants