Skip to content
This repository was archived by the owner on Jul 28, 2025. It is now read-only.

Fix cudf::column constructor args#1151

Merged
rapids-bot[bot] merged 5 commits intorapidsai:branch-23.06from
trxcllnt:fix/cudf-column-ctor
May 26, 2023
Merged

Fix cudf::column constructor args#1151
rapids-bot[bot] merged 5 commits intorapidsai:branch-23.06from
trxcllnt:fix/cudf-column-ctor

Conversation

@trxcllnt
Copy link
Copy Markdown
Contributor

@trxcllnt trxcllnt commented May 23, 2023

Description

Update cudf::column constructor args to match the changes in rapidsai/cudf#13341.

Also corrects a minor issue in the docs, closes #1154.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@trxcllnt trxcllnt added 3 - Ready for Review Ready for review by team improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels May 23, 2023
@trxcllnt trxcllnt requested a review from a team as a code owner May 23, 2023 21:47
@trxcllnt trxcllnt requested review from harrism and isVoid May 23, 2023 21:47
@github-actions github-actions bot added the libcuspatial Relates to the cuSpatial C++ library label May 23, 2023
@trxcllnt
Copy link
Copy Markdown
Contributor Author

Failing test seems like an unrelated cuML bug.

@isVoid
Copy link
Copy Markdown
Contributor

isVoid commented May 24, 2023

Failing test seems like an unrelated cuML bug.

Tracked by: rapidsai/cuml#5434

@harrism
Copy link
Copy Markdown
Member

harrism commented May 24, 2023

How do we rerun tests? Is the cuML fix in last night's nightly?

@harrism
Copy link
Copy Markdown
Member

harrism commented May 24, 2023

How do we rerun tests? Is the cuML fix in last night's nightly?

FIgured this out. Click on failing action, look for rerun icon in top right (circular arrows).
image

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@bdice
Copy link
Copy Markdown
Contributor

bdice commented May 26, 2023

/merge

@rapids-bot rapids-bot bot merged commit 5162e39 into rapidsai:branch-23.06 May 26, 2023
@bdice
Copy link
Copy Markdown
Contributor

bdice commented May 26, 2023

I went ahead and merged this since it fixes nightlies, which are needed for 23.06 release testing. cc: @beckernick this should be good to go once builds finish on the merge commit and are pushed to rapidsai-nightly.

@trxcllnt trxcllnt deleted the fix/cudf-column-ctor branch May 26, 2023 20:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

3 - Ready for Review Ready for review by team improvement Improvement / enhancement to an existing function libcuspatial Relates to the cuSpatial C++ library non-breaking Non-breaking change

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

5 participants