Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CUDA: fix log size #3191

Merged
merged 1 commit into from
Aug 2, 2018
Merged

CUDA: fix log size #3191

merged 1 commit into from
Aug 2, 2018

Conversation

njwhite
Copy link
Contributor

@njwhite njwhite commented Jul 31, 2018

This throws on line 874 if you set the environ variable, as you can't multiply a string. I don't think it ever worked!

@stuartarchibald stuartarchibald added the CUDA CUDA related issue/PR label Jul 31, 2018
@codecov-io
Copy link

codecov-io commented Aug 1, 2018

Codecov Report

Merging #3191 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3191   +/-   ##
=======================================
  Coverage   81.08%   81.08%           
=======================================
  Files         384      384           
  Lines       75186    75186           
  Branches     8440     8440           
=======================================
  Hits        60964    60964           
  Misses      12931    12931           
  Partials     1291     1291

Copy link
Member

@sklam sklam left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@sklam sklam merged commit 20ec0bd into numba:master Aug 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review CUDA CUDA related issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants