Skip to content

Use Legion with CMake's native CUDA language#828

Merged
manopapad merged 10 commits intonv-legate:branch-23.09from
trxcllnt:fix/legion-cmake-cuda-update
Sep 1, 2023
Merged

Use Legion with CMake's native CUDA language#828
manopapad merged 10 commits intonv-legate:branch-23.09from
trxcllnt:fix/legion-cmake-cuda-update

Conversation

@trxcllnt
Copy link
Contributor

Redo of #725 that touches fewer CI-related files. Updates Legion to the commit hash in #827.

@magnatelee magnatelee added the category:improvement PR introduces an improvement and will be classified as such in release notes label Aug 30, 2023
@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Aug 31, 2023

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.

@trxcllnt
Copy link
Contributor Author

/ok to test

@trxcllnt trxcllnt force-pushed the fix/legion-cmake-cuda-update branch from 119508e to e947e42 Compare August 31, 2023 02:38
@Jacobfaib
Copy link
Contributor

@trxcllnt If not already, can you also add a CI job(s) that tests the editable install + venv in this PR?

@trxcllnt
Copy link
Contributor Author

trxcllnt commented Sep 1, 2023

@Jacobfaib That's going to be a bit of a stretch, as this PR is rather urgent to unblock folks who need to use the latest control_replication. I plan on following up with additional workflow PRs once we finalize devcontainers in the next few weeks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:improvement PR introduces an improvement and will be classified as such in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants