[pre-commit.ci] pre-commit autoupdate #1108

pre-commit-ci · 2024-01-01T19:51:23Z

updates:

copy-pr-bot · 2024-01-01T19:51:27Z

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

manopapad · 2024-01-02T19:58:30Z

/ok to test

* Fixing build issues * Bump the Legate commit hash to run CI

* fix compilation errors with cuda_help.h * cuda_help.h: wrap check_* in namespace cunumeric * fix cuda_help: add cunumeric:: in the macro --------- Co-authored-by: Wonchan Lee <[email protected]>

* installation chapter * examples to their own chapter * comparison to API ref * howto api measure * add shrirams new examples * faq (empty) structure * more user guide re-org * add shriram's usage changes * more of shriram's content * finish faqs * remove old configuration chapter * review comments

* Build C++ tests as part of main build * Add install.py option * fixups * bump legate commit

* remove pytest_lazy_fixture dependence * more pytest8 incompat

* Test against older legate version numbering * Check against the recently merged legate nightly version.

* Try enabling arm64 packages * Restrict to numpy 1.x --------- Co-authored-by: Manolis Papadakis <[email protected]>

* enhance coverage for _array/*.py

Co-authored-by: Mark Vaz <[email protected]>

The conda install will take care of this automatically, and this list will almost certainly bitrot

* prioritize demonstrating gpu options * update practices headings * Apply suggestions from code review Co-authored-by: Manolis Papadakis <[email protected]> --------- Co-authored-by: Manolis Papadakis <[email protected]>

This updates the build scripts to build conda packages using the new format.

* enhance test_flags.py

* Add TaskConfig support, see nv-legate/legate.internal#1895

Also pet-peeve; rename all static-registration variables to align

Ignoring known issues.

This is due to limited availability of ARM64 GPU runners for the project.

Also ... - Use [email protected]. - Use a cpu16 machine to build cupynumeric even when building docs.

* Update for removal of LEGATE_NEED_* envvars * Update to latest legate * add channel for UCC 1.4.0 * one more ucc140 * typo --------- Co-authored-by: Marcin Zalewski <[email protected]>

This switches to using artifact upload/download to pass the wheel from build to test jobs. It removes some debug and pins the action being used for inter-repository artifact download to the v9 release SHA.

Add some utility scripts to use the official `gh` CLI to look up the run ID and download artifacts. This enables us to migrate away from the third-party action and offers a base of capabilities as we split the conda packages and pip wheels. This is inspired by and shamelessly takes from https://github.com/rapidsai/gha-tools/, specifically `rapids-download-from-github` and `rapids-github-run-id` adapting them for these projects.

The version file is ironically causing the version string to be incorrect. Issue #666 documents what should be done to fix this correctly. In the short term I need a clean release version for the upcoming initial wheels release hence overriding it. Longer term we should override in CI anyway on the release branch to switch to a post release versioning scheme in release branches.

Push source code to nv-legate/cupynumeric as part of the nightly CI run. Tracking issue: LLRDO-335.

Get the Legate SHA from the `cmake/versions.cmake` file for the wheels, this is the same place that the conda-based CI/build gets the SHA from creating a single source of truth. Also bump the legate wheel dependency to `25.5.*` for `main`.

The git SHA and build number should be at the end of the build string, the build number is especially important in assisting the solver in resolving the latest version that should be installed.

Enable `sccache` for the wheels and conda builds of the project. This yielded ball park improvements of: - 16-19 minutes down to 3-5 minutes for wheel builds - 28 minutes down to 7 minutes for conda GPU builds - 10-11 minutes down to 4 minutes for conda CPU builds No observed regressions in the tests.

* add special case to handle matmul ufunc * Update cupynumeric/_array/array.py Co-authored-by: Manolis Papadakis <[email protected]> --------- Co-authored-by: Manolis Papadakis <[email protected]>

updates: - [github.com/pre-commit/mirrors-mypy: v1.5.1 → v1.15.0](pre-commit/mirrors-mypy@v1.5.1...v1.15.0) - [github.com/PyCQA/isort: 5.12.0 → 6.0.1](PyCQA/isort@5.12.0...6.0.1) - [github.com/psf/black: 23.9.1 → 25.1.0](psf/black@23.9.1...25.1.0) - [github.com/PyCQA/flake8: 6.1.0 → 7.2.0](PyCQA/flake8@6.1.0...7.2.0) - [github.com/pre-commit/mirrors-clang-format: v16.0.6 → v20.1.0](pre-commit/mirrors-clang-format@v16.0.6...v20.1.0)

for more information, see https://pre-commit.ci

manopapad approved these changes Jan 2, 2024

View reviewed changes

manopapad added the category:task PR is a simple task and will not be included in release notes label Jan 4, 2024

marcinz changed the base branch from branch-24.01 to branch-24.03 February 22, 2024 01:03

pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from ccdcbb8 to 9cfc501 Compare April 1, 2024 21:32

Jacobfaib and others added 24 commits June 6, 2024 21:02

Fix include order fiasco (#220)

2a1a1bc

Fixing build issues (#223)

bb35036

* Fixing build issues * Bump the Legate commit hash to run CI

Revert a recent fix, which wasn't the right fix (#224)

fb7b80e

Fix CI issues (#222)

76060c0

Fix compilation errors with cuda_help.h (#225)

1ae7bcc

* fix compilation errors with cuda_help.h * cuda_help.h: wrap check_* in namespace cunumeric * fix cuda_help: add cunumeric:: in the macro --------- Co-authored-by: Wonchan Lee <[email protected]>

Fix for the CPU build (#226)

0cb9f2d

Build C++ tests as part of main build (#218)

6e01c34

* Build C++ tests as part of main build * Add install.py option * fixups * bump legate commit

remove pytest_lazy_fixture dependency (#229)

f56a67a

* remove pytest_lazy_fixture dependence * more pytest8 incompat

Update legate version (#232)

8eb1a7f

* Test against older legate version numbering * Check against the recently merged legate nightly version.

Try enabling arm64 packages (#233)

1a09a7f

* Try enabling arm64 packages * Restrict to numpy 1.x --------- Co-authored-by: Manolis Papadakis <[email protected]>

Merge branch 'cpp-branch-24.05'

9eafdea

conditionally use the nv theme with NV_THEME=1 (#234)

54cd6d9

enhance coverage for _array/*.py (#230)

1e45281

* enhance coverage for _array/*.py

Trigger workflow for main branch. (#237)

cf2cad0

Co-authored-by: Mark Vaz <[email protected]>

Update Legate (#236)

fde89d3

Build aarch64 (#241)

6b4e342

Update CI (#248)

2793827

Change package tagging (#246)

f108bc0

Fix package build (#250)

540a9ed

only match np exception type (#239)

65422e0

enhance coverage for _module/stats_order.py (#238)

8bda90f

Remove list of requirements (#240)

8a90b95

The conda install will take care of this automatically, and this list will almost certainly bitrot

Misc docs updates (#253)

ecc560c

* prioritize demonstrating gpu options * update practices headings * Apply suggestions from code review Co-authored-by: Manolis Papadakis <[email protected]> --------- Co-authored-by: Manolis Papadakis <[email protected]>

cryos and others added 21 commits March 7, 2025 12:39

Update to build new format conda packages (#577)

78997f8

This updates the build scripts to build conda packages using the new format.

enhance test_flags.py (#640)

eb4e972

* enhance test_flags.py

Bump to 25.05 (#643)

079679b

Add TaskConfig support (#641)

da8c96f

* Add TaskConfig support, see nv-legate/legate.internal#1895

updating legate commit hash (#648)

e7356d3

Add separate docs workflow and push nightly docs to gh pages. (#611)

f22ae8c

Fixes #645 (#647)

3f1e884

Also pet-peeve; rename all static-registration variables to align

Fix nvbug 5156643 (#649)

ab44cbf

Documentation fixes (#651)

9dba1f3

Fix build has tests (#652) (#653)

4298c00

CPP tests (#447)

7569d68

Ignoring known issues.

Disable ARM64 GPU testing for wheels (#655)

a29a2cf

This is due to limited availability of ARM64 GPU runners for the project.

Upload docs only on main (#657)

bdceb14

Also ... - Use [email protected]. - Use a cpu16 machine to build cupynumeric even when building docs.

Update for removal of LEGATE_NEED_* envvars (#650)

06244e4

* Update for removal of LEGATE_NEED_* envvars * Update to latest legate * add channel for UCC 1.4.0 * one more ucc140 * typo --------- Co-authored-by: Marcin Zalewski <[email protected]>

Remove debug and don't cache the wheel (#660)

3e88ba2

This switches to using artifact upload/download to pass the wheel from build to test jobs. It removes some debug and pins the action being used for inter-repository artifact download to the v9 release SHA.

Update licenses that were missed - wheels (#664)

1f53931

Nightly push to external repo (#659)

a55045d

Push source code to nv-legate/cupynumeric as part of the nightly CI run. Tracking issue: LLRDO-335.

Get the Legate SHA from versions.cmake (#669)

73f64bb

Get the Legate SHA from the `cmake/versions.cmake` file for the wheels, this is the same place that the conda-based CI/build gets the SHA from creating a single source of truth. Also bump the legate wheel dependency to `25.5.*` for `main`.

Move the conda build string version info to end (#671)

7fc9dc9

The git SHA and build number should be at the end of the build string, the build number is especially important in assisting the solver in resolving the latest version that should be installed.

manopapad force-pushed the branch-24.03 branch from 3d14ed5 to a8259c5 Compare April 2, 2025 21:09

marcinz and others added 6 commits April 3, 2025 00:08

Update runners to NVKS (#588)

45492a8

Update CODEOWNERS to use the GitHub team (#675)

70e91d5

Wheels docs (#670)

0a1f3dc

add special case to handle matmul ufunc (#679)

3715044

* add special case to handle matmul ufunc * Update cupynumeric/_array/array.py Co-authored-by: Manolis Papadakis <[email protected]> --------- Co-authored-by: Manolis Papadakis <[email protected]>

pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 94c8688 to a26f8fc Compare April 7, 2025 20:03

[pre-commit.ci] auto fixes from pre-commit.com hooks

2f2a4d3

for more information, see https://pre-commit.ci

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[pre-commit.ci] pre-commit autoupdate #1108

[pre-commit.ci] pre-commit autoupdate #1108

Uh oh!

pre-commit-ci bot commented Jan 1, 2024 •

edited

Loading

Uh oh!

copy-pr-bot bot commented Jan 1, 2024

Uh oh!

manopapad commented Jan 2, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

[pre-commit.ci] pre-commit autoupdate #1108

Are you sure you want to change the base?

[pre-commit.ci] pre-commit autoupdate #1108

Uh oh!

Conversation

pre-commit-ci bot commented Jan 1, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

copy-pr-bot bot commented Jan 1, 2024

Uh oh!

manopapad commented Jan 2, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

pre-commit-ci bot commented Jan 1, 2024 •

edited

Loading