Skip to content

Use libcufile in aarch64 CUDA 12 builds.#330

Closed
bdice wants to merge 1 commit intorapidsai:branch-24.02from
bdice:aarch64-cuda12-libcufile
Closed

Use libcufile in aarch64 CUDA 12 builds.#330
bdice wants to merge 1 commit intorapidsai:branch-24.02from
bdice:aarch64-cuda12-libcufile

Conversation

@bdice
Copy link
Copy Markdown
Contributor

@bdice bdice commented Jan 16, 2024

CUDA 12.2 has conda-forge packages of libcufile and libcufile-dev. This PR updates dependencies.yaml so that environments can be generated on ARM.

@jakirkham jakirkham added improvement Improves an existing functionality non-breaking Introduces a non-breaking change conda labels Jan 16, 2024
@jakirkham
Copy link
Copy Markdown
Member

Thanks Bradley! 🙏

Do we want to update lines like these as well?

{% if cuda_major == "11" %}
- cudatoolkit
- libcufile {{ cuda11_libcufile_host_version }} # [linux64]
- libcufile-dev {{ cuda11_libcufile_host_version }} # [linux64]
{% else %}
- libcufile-dev # [linux64]
{% endif %}

@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Jan 16, 2024

@jakirkham Good question. I think I would prefer to put those recipe changes in #328, since we aren't enabling CUDA 12.2 workflows yet. Maybe this PR should be closed and ported into #328...?

@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Jan 16, 2024

I'm closing this in favor of merging changes to #328, which will include the conda recipe changes for libcufile.

@bdice bdice closed this Jan 16, 2024
@jakirkham
Copy link
Copy Markdown
Member

jakirkham commented Jan 16, 2024

No strong feelings either way

@jameslamb what do you think?

Edit: Oops not fast enough 😅 James could you please pull these changes into your PR?

Edit 2: Maybe this already happened ( edb09af )?

@jameslamb
Copy link
Copy Markdown
Member

ha no prob, yeah looks like @bdice pulled them into #328 already

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

Labels

conda improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants