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.

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

Make current matrix work nested under nvcc field #240

Closed
Tracked by #239
jrhemstad opened this issue Jul 18, 2023 · 0 comments · Fixed by #243
Closed
Tracked by #239

Make current matrix work nested under nvcc field #240

jrhemstad opened this issue Jul 18, 2023 · 0 comments · Fixed by #243
Assignees

Comments

@jrhemstad
Copy link
Collaborator

jrhemstad commented Jul 18, 2023

As part of #239, we want to nest the current matrix environment definitions under a nvcc: field and make the workflow work.

We'll primarily need to update the compute-matrix reusable workflow to reference the nvcc: field. The outputs of this workflow also presuppose an nvcc job, so we'll need to figure out how to generalize the outputs:

outputs:
FULL_MATRIX:
description: "The full matrix of build configurations for the given matrix type."
value: ${{ jobs.compute-matrix.outputs.FULL_MATRIX }}
CUDA_VERSIONS:
description: "The unique CUDA versions in the full matrix."
value: ${{ jobs.compute-matrix.outputs.CUDA_VERSIONS }}
COMPILERS:
description: "The unique compilers in the full matrix."
value: ${{ jobs.compute-matrix.outputs.COMPILERS }}
PER_CUDA_COMPILER_MATRIX:
description: "The matrix of build configurations for each CUDA version and compiler."
value: ${{ jobs.compute-matrix.outputs.PER_CUDA_COMPILER_MATRIX }}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant