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.

Sign up for GitHub

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

Jump to bottom

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

Merged
merged 2 commits into from
May 3, 2024
Merged

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

merged 2 commits into from
May 3, 2024

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Mar 12, 2024 •
edited by csbnw
Loading

updates:

This update failed because the workflow used the cmakelang package, while the pre-commit config uses clang-format and cmake-format.

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 7846ad5 to ed0d7b8 Compare March 25, 2024 19:28
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from ed0d7b8 to ff8fb67 Compare April 8, 2024 19:47
@pre-commit-ci
[pre-commit.ci] pre-commit autoupdate
6d72bb9 
updates:
- [github.com/pre-commit/mirrors-clang-format: v17.0.6 → v18.1.4](pre-commit/mirrors-clang-format@v17.0.6...v18.1.4)
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from ff8fb67 to 6d72bb9 Compare April 22, 2024 19:29
@csbnw
Replace cmakelang with clang-format and cmake-format
e5be576 
This matches the .pre-commit-config.yaml
@csbnw csbnw requested a review from loostrum May 3, 2024 08:22
@csbnw csbnw merged commit 2600cf1 into main May 3, 2024
7 checks passed
@csbnw csbnw deleted the pre-commit-ci-update-config branch May 3, 2024 08:24
loostrum added a commit that referenced this pull request Jun 18, 2024
@loostrum @john-romein
@csbnw @pre-commit-ci
Merge main into hipified (#282)
6ef7d19 
* Added cu::Function::occupancyMaxActiveBlocksPerMultiprocessor()

* Let target_embed_source inline local header files (#263)

* Fix working directory for linking source into object file (#264)

* Update readthedocs configuration (#265)

* Fix target_embed_source when input file is a relative path (#266)

* Removed deprecated cu::Context::setSharedMemConfig (#267)

* Extend api (#268)

* Add Device::getUUID
* Add initial nvml wrapper

* Fix target_embed_source (#269)

* Fix target embed source paths (#270)

* Fix path given to ld in target_embed_source
* Fix missing arg in string replace

* Fix CI (#271)

* Add runtime subpackage
* Fix symbols for test_nvrtc

* [pre-commit.ci] pre-commit autoupdate (#259)

* Update clang-format: v17.0.6 → v18.1.4
* Replace cmakelang with clang-format and cmake-format

* Add nvrtc::findIncludePath (#272)

* Make nvrtc::findIncludePath inline (#273)

* Fix string replacements (#275)

* Fix inline_local_includes (#276)

* Make sure to select the correct INCLUDE_PATH (#277)

* Fix typo (#278)

* [pre-commit.ci] pre-commit autoupdate (#274)

updates:
- [github.com/pre-commit/mirrors-clang-format: v18.1.4 → v18.1.5](pre-commit/mirrors-clang-format@v18.1.4...v18.1.5)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* Add inlined headers as dependencies in target_embed_source (#279)

* Add inlined headers as dependencies in target_embed_source

This ensure that the embedded sources are rebuild when a header is changed

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

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

* Update cmake/cudawrappers-helper.cmake

Co-authored-by: Bram Veenboer <veenboer@astron.nl>

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Bram Veenboer <veenboer@astron.nl>

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

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

* Add HIP version of occupancyMaxActiveBlocksPerMultiprocessor

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

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

---------

Co-authored-by: John Romein <romein@astron.nl>
Co-authored-by: john-romein <39552858+john-romein@users.noreply.github.com>
Co-authored-by: Bram Veenboer <veenboer@astron.nl>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
csbnw added a commit that referenced this pull request Jul 9, 2024
@loostrum @john-romein
@csbnw @pre-commit-ci
Merge main into hipified (#282)
ab147ad 
* Added cu::Function::occupancyMaxActiveBlocksPerMultiprocessor()

* Let target_embed_source inline local header files (#263)

* Fix working directory for linking source into object file (#264)

* Update readthedocs configuration (#265)

* Fix target_embed_source when input file is a relative path (#266)

* Removed deprecated cu::Context::setSharedMemConfig (#267)

* Extend api (#268)

* Add Device::getUUID
* Add initial nvml wrapper

* Fix target_embed_source (#269)

* Fix target embed source paths (#270)

* Fix path given to ld in target_embed_source
* Fix missing arg in string replace

* Fix CI (#271)

* Add runtime subpackage
* Fix symbols for test_nvrtc

* [pre-commit.ci] pre-commit autoupdate (#259)

* Update clang-format: v17.0.6 → v18.1.4
* Replace cmakelang with clang-format and cmake-format

* Add nvrtc::findIncludePath (#272)

* Make nvrtc::findIncludePath inline (#273)

* Fix string replacements (#275)

* Fix inline_local_includes (#276)

* Make sure to select the correct INCLUDE_PATH (#277)

* Fix typo (#278)

* [pre-commit.ci] pre-commit autoupdate (#274)

updates:
- [github.com/pre-commit/mirrors-clang-format: v18.1.4 → v18.1.5](pre-commit/mirrors-clang-format@v18.1.4...v18.1.5)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* Add inlined headers as dependencies in target_embed_source (#279)

* Add inlined headers as dependencies in target_embed_source

This ensure that the embedded sources are rebuild when a header is changed

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

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

* Update cmake/cudawrappers-helper.cmake

Co-authored-by: Bram Veenboer <veenboer@astron.nl>

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Bram Veenboer <veenboer@astron.nl>

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

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

* Add HIP version of occupancyMaxActiveBlocksPerMultiprocessor

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

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

---------

Co-authored-by: John Romein <romein@astron.nl>
Co-authored-by: john-romein <39552858+john-romein@users.noreply.github.com>
Co-authored-by: Bram Veenboer <veenboer@astron.nl>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@loostrum loostrum loostrum approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@loostrum @csbnw