Skip to content

3335 Fix hash verification of Clang#3337

Closed
Nic-Ma wants to merge 8 commits into
Project-MONAI:devfrom
Nic-Ma:3335-fix-clang-test
Closed

3335 Fix hash verification of Clang#3337
Nic-Ma wants to merge 8 commits into
Project-MONAI:devfrom
Nic-Ma:3335-fix-clang-test

Conversation

@Nic-Ma

@Nic-Ma Nic-Ma commented Nov 15, 2021

Copy link
Copy Markdown
Contributor

Fixes #3335 .

Description

This PR fixed the hash verification of Clang format package, I think they updated the binary file in Amazon S3.

Status

Ready

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Breaking change (fix or new feature that would cause existing functionality to change).
  • New tests added to cover the changes.
  • Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests.
  • In-line docstrings updated.
  • Documentation updated, tested make html command in the docs/ folder.

@Nic-Ma

Nic-Ma commented Nov 15, 2021

Copy link
Copy Markdown
Contributor Author

/black

@Nic-Ma

Nic-Ma commented Nov 15, 2021

Copy link
Copy Markdown
Contributor Author

/build

@Nic-Ma Nic-Ma requested review from ericspod, rijobro and wyli November 15, 2021 02:47
Signed-off-by: Nic Ma <nma@nvidia.com>
@Nic-Ma Nic-Ma force-pushed the 3335-fix-clang-test branch from 5f57ce5 to ca21c6d Compare November 15, 2021 02:48
Signed-off-by: Nic Ma <nma@nvidia.com>
Signed-off-by: Nic Ma <nma@nvidia.com>
@Nic-Ma

Nic-Ma commented Nov 15, 2021

Copy link
Copy Markdown
Contributor Author

/black

@Nic-Ma

Nic-Ma commented Nov 15, 2021

Copy link
Copy Markdown
Contributor Author

/build

@Nic-Ma

Nic-Ma commented Nov 15, 2021

Copy link
Copy Markdown
Contributor Author

/build

@wyli

wyli commented Nov 15, 2021

Copy link
Copy Markdown
Contributor

Hi @Nic-Ma, this has been updated to use sha256 in the pytorch repo https://github.com/pytorch/pytorch/blob/1adeeabdc0c8832420c091c5c668843768530d7f/tools/linter/adapters/clangtidy_init.py#L128 I recommend we use #3340.

@Nic-Ma

Nic-Ma commented Nov 15, 2021

Copy link
Copy Markdown
Contributor Author

Hi @wyli ,

OK, looks good to me.
I will close this PR.

Thanks.

@Nic-Ma Nic-Ma closed this Nov 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

unit test error from python -m tests.clang_format_utils

2 participants