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

[REVIEW] Invalid children check in mutable_column_device_view #3280

Merged

Conversation

davidwendt
Copy link
Contributor

Children are not supported for mutable_column_device_view. The CUDF_EXPECTS check for this was incorrect and fixed by this PR.

@davidwendt davidwendt requested a review from a team as a code owner November 1, 2019 16:13
@davidwendt davidwendt self-assigned this Nov 1, 2019
@davidwendt davidwendt added 3 - Ready for Review Ready for review by team libcudf++ labels Nov 1, 2019
@codecov
Copy link

codecov bot commented Nov 1, 2019

Codecov Report

Merging #3280 into branch-0.11 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           branch-0.11    #3280   +/-   ##
============================================
  Coverage        87.13%   87.13%           
============================================
  Files               49       49           
  Lines             9269     9269           
============================================
  Hits              8077     8077           
  Misses            1192     1192

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e7bece2...7b881dc. Read the comment docs.

@jrhemstad jrhemstad merged commit 3c9f011 into rapidsai:branch-0.11 Nov 1, 2019
@davidwendt davidwendt deleted the bug-fix-mutable-children branch November 8, 2019 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review Ready for review by team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants