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

[gpuCI] Forward-merge branch-21.10 to branch-21.12 [skip gpuci] #4349

Merged
merged 7 commits into from
Nov 17, 2021

Conversation

GPUtester
Copy link
Contributor

Forward-merge triggered by push to branch-21.10 that creates a PR to keep branch-21.12 up-to-date. If this PR is unable to be immediately merged due to conflicts, it will remain open for the team to manually merge.

ajschmidt8 and others added 3 commits November 10, 2021 11:12
This PR updates the flags for to ignore certain warnings from being treated as errors. This is a necessary hotfix to get DLFW builds to complete successfully.
…provided to FIL model (#4315)

Fix potential CUDA context poison due to invalid global read when negative categories provided at inference:

now equivalent to non-matching. Same for `+-Inf` categories. NAN categories are still `!def_left`, and fractional categories are truncated (via a typecast), as this is what Treelite does

FIL now converts dummy nodes to numerical on import
and never generates max_matching == -1 categorical features in test.
FIL will still generate empty categorical nodes in test (a non-empty bits vector which contains only zeros), export them as dummy numerical nodes and import again as dummy numerical nodes. If a feature only contains dummy numerical nodes, it will be deemed a numerical feature (same as for non-dummy numerical nodes or a mix thereof). Therefore, categorical feature max_matching == -1 is still prevented.
@GPUtester GPUtester requested review from a team as code owners November 10, 2021 21:19
@GPUtester
Copy link
Contributor Author

FAILURE - Unable to forward-merge due to conflicts, manual merge is necessary. Do not use the Resolve conflicts option in this PR, follow these instructions https://docs.rapids.ai/maintainers/gpuci/#forward-mergers
IMPORTANT: Before merging and approving this PR, be sure to change the merging strategy to Create a Merge Commit. Otherwise, history will be lost and the branches become incompatible.

@github-actions github-actions bot added CMake CUDA/C++ Cython / Python Cython or Python issue gpuCI gpuCI issue labels Nov 10, 2021
ajschmidt8 added a commit that referenced this pull request Nov 10, 2021
This PR fixes the logic for when the `nightly` channel should be removed.

The change is also incorporated in the forward-merger PR below, but it may take a while to resolve the conflicts there.

#4349

Authors:
  - AJ Schmidt (https://github.com/ajschmidt8)

Approvers:
  - Ray Douglass (https://github.com/raydouglass)
  - Jordan Jacobelli (https://github.com/Ethyling)

URL: #4350
Copy link
Member

@dantegd dantegd left a comment

Choose a reason for hiding this comment

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

Leaving a comment: the forward merger doesn't need to be addressed to branch-21.12, different PRs will add improve versions of the fixes to 21.12

ajschmidt8 added a commit that referenced this pull request Nov 17, 2021
@ajschmidt8 ajschmidt8 merged commit 5d81fc5 into branch-21.12 Nov 17, 2021
vimarsh6739 pushed a commit to vimarsh6739/cuml that referenced this pull request Oct 9, 2023
This PR fixes the logic for when the `nightly` channel should be removed.

The change is also incorporated in the forward-merger PR below, but it may take a while to resolve the conflicts there.

rapidsai#4349

Authors:
  - AJ Schmidt (https://github.com/ajschmidt8)

Approvers:
  - Ray Douglass (https://github.com/raydouglass)
  - Jordan Jacobelli (https://github.com/Ethyling)

URL: rapidsai#4350
vimarsh6739 pushed a commit to vimarsh6739/cuml that referenced this pull request Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CMake CUDA/C++ Cython / Python Cython or Python issue gpuCI gpuCI issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants