Skip to content

Updating Clang Version to 11.0.0 - #4029

Merged
rapids-bot[bot] merged 14 commits into
NVIDIA:branch-21.08from
codereport:clang-11
Jul 9, 2021
Merged

Updating Clang Version to 11.0.0#4029
rapids-bot[bot] merged 14 commits into
NVIDIA:branch-21.08from
codereport:clang-11

Conversation

@codereport

@codereport codereport commented Jul 5, 2021

Copy link
Copy Markdown
Contributor

Follow up PR to: NVIDIA/cudf#6695. Performing the same changes but for rapidsai/cuml

Depends on: rapidsai/integration#304

@codereport
codereport requested a review from a team as a code owner July 5, 2021 00:24
@codereport codereport mentioned this pull request Jul 5, 2021
Merged
@codereport
codereport requested a review from a team as a code owner July 5, 2021 14:26
@github-actions github-actions Bot added the gpuCI gpuCI issue label Jul 5, 2021
@ajschmidt8 ajschmidt8 added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 6, 2021
@codereport

codereport commented Jul 6, 2021

Copy link
Copy Markdown
Contributor Author

rerun tests

@github-actions github-actions Bot removed the gpuCI gpuCI issue label Jul 6, 2021
@codereport

codereport commented Jul 6, 2021

Copy link
Copy Markdown
Contributor Author

rerun tests

wphicks
wphicks approved these changes Jul 6, 2021
View reviewed changes

@wphicks wphicks left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I went over everything in here fairly quickly and don't see any problems from a technical standpoint. The only question I have is whether there should be some cross-team discussion before we standardize on exactly this set of formatting rules. We're reversing or modifying several rules we had in cuML, so I want to make sure that everyone is onboard with that if we're looking to standardize across projects.

I'm giving my approval on the basis of the functional changes here, and I've prompted the cuML folks internally to weigh in if they care about this.

@codereport

codereport commented Jul 7, 2021

Copy link
Copy Markdown
Contributor Author

rerun tests

5 similar comments
@codereport

codereport commented Jul 7, 2021

Copy link
Copy Markdown
Contributor Author

rerun tests

@codereport

codereport commented Jul 7, 2021

Copy link
Copy Markdown
Contributor Author

rerun tests

@ajschmidt8

ajschmidt8 commented Jul 7, 2021

Copy link
Copy Markdown
Member

rerun tests

@codereport

codereport commented Jul 7, 2021

Copy link
Copy Markdown
Contributor Author

rerun tests

@ajschmidt8

ajschmidt8 commented Jul 8, 2021

Copy link
Copy Markdown
Member

rerun tests

@wphicks

wphicks commented Jul 8, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor

This will not pass until #3990 is in, so no need to rerun tests until that happens and this PR is updated with those changes.

@github-actions github-actions Bot added CMake Cython / Python Cython or Python issue labels Jul 8, 2021
@codereport
Revert "Merge 21.08"
ab5cc69 
This reverts commit 490be85.
@codereport
codereport requested review from a team as code owners July 8, 2021 18:30
@github-actions github-actions Bot added gpuCI gpuCI issue and removed CMake Cython / Python Cython or Python issue labels Jul 8, 2021
@codecov-commenter

codecov-commenter commented Jul 8, 2021

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (branch-21.08@954d7cb). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##             branch-21.08    #4029   +/-   ##
===============================================
  Coverage                ?   85.46%           
===============================================
  Files                   ?      230           
  Lines                   ?    18139           
  Branches                ?        0           
===============================================
  Hits                    ?    15502           
  Misses                  ?     2637           
  Partials                ?        0
Flag Coverage Δ
dask 48.14% <0.00%> (?)
non-dask 77.74% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 954d7cb...9f88b39. Read the comment docs.

@cjnolet

cjnolet commented Jul 9, 2021

Copy link
Copy Markdown
Collaborator

@gpucibot merge

@rapids-bot
rapids-bot Bot merged commit bcc4cad into NVIDIA:branch-21.08 Jul 9, 2021
@vyasr vyasr mentioned this pull request Jul 9, 2021
Closed
vimarsh6739 pushed a commit to vimarsh6739/cuml that referenced this pull request Oct 9, 2023
@codereport
Updating Clang Version to 11.0.0 (NVIDIA#4029)
1db7e9d 
Follow up PR to: NVIDIA/cudf#6695. Performing the same changes but for `rapidsai/cuml`

Depends on: rapidsai/integration#304

Authors:
  - Conor Hoekstra (https://github.com/codereport)

Approvers:
  - William Hicks (https://github.com/wphicks)
  - AJ Schmidt (https://github.com/ajschmidt8)
  - Robert Maynard (https://github.com/robertmaynard)
  - Dante Gama Dessavre (https://github.com/dantegd)

URL: NVIDIA#4029
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@robertmaynard robertmaynard robertmaynard approved these changes
@dantegd dantegd dantegd approved these changes
@ajschmidt8 ajschmidt8 ajschmidt8 approved these changes
+1 more reviewer
@wphicks wphicks wphicks approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

CUDA/C++ gpuCI gpuCI issue improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@codereport @ajschmidt8 @wphicks @codecov-commenter @cjnolet @robertmaynard @dantegd @caryr35