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

cusparselt v0.7.0.0 #52

Merged
merged 3 commits into from
Jan 29, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .azure-pipelines/azure-pipelines-linux.yml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
c_compiler:
- gcc
c_compiler_version:
- '12'
- '13'
c_stdlib:
- sysroot
c_stdlib_version:
- '2.17'
- '2.28'
cdt_name:
- conda
channel_sources:
Expand All @@ -19,7 +19,7 @@ cuda_compiler_version:
cxx_compiler:
- gxx
cxx_compiler_version:
- '12'
- '13'
docker_image:
- quay.io/condaforge/linux-anvil-x86_64:alma9
target_platform:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
c_compiler:
- gcc
c_compiler_version:
- '12'
- '13'
c_stdlib:
- sysroot
c_stdlib_version:
- '2.17'
- '2.28'
cdt_name:
- conda
channel_sources:
Expand All @@ -19,7 +19,7 @@ cuda_compiler_version:
cxx_compiler:
- gxx
cxx_compiler_version:
- '12'
- '13'
docker_image:
- quay.io/condaforge/linux-anvil-aarch64:alma9
target_platform:
Expand Down
3 changes: 3 additions & 0 deletions .gitignore

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 4 additions & 4 deletions README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 3 additions & 2 deletions recipe/conda_build_config.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@

arm_variant_type: # [aarch64]
- sbsa # [aarch64]
c_stdlib_version: # [linux]
- "2.17" # [linux]
- "2.28" # [linux]
21 changes: 10 additions & 11 deletions recipe/meta.yaml
Original file line number Diff line number Diff line change
@@ -1,19 +1,19 @@
{% set version = "0.6.3.2" %}
{% set version = "0.7.0.0" %}

package:
name: cusparselt
version: {{ version }}

source:
- url: https://developer.download.nvidia.com/compute/cusparselt/redist/libcusparse_lt/linux-x86_64/libcusparse_lt-linux-x86_64-{{ version }}-archive.tar.xz # [linux64]
sha256: a2f856e78943f5c538bdef1c9edc64a5ed30bf8bb7d5fcb615c684ffe776cc31 # [linux64]
sha256: 6082f3d7f024b0183225d48361ca8436fdf30d5833e3595c6c145c03171c13ee # [linux64]
- url: https://developer.download.nvidia.com/compute/cusparselt/redist/libcusparse_lt/linux-sbsa/libcusparse_lt-linux-sbsa-{{ version }}-archive.tar.xz # [aarch64]
sha256: 3e420ddbff4eb9ac603f57c7aa8b3d5271112816e244eb55ef9f30c4eb6a04b7 # [aarch64]
sha256: f625038f52dec7068824889d523e16112a2363c7e4f1da87adaac6db6bb7caa5 # [aarch64]
- url: https://developer.download.nvidia.com/compute/cusparselt/redist/libcusparse_lt/windows-x86_64/libcusparse_lt-windows-x86_64-{{ version }}-archive.zip # [win64]
sha256: 6d276e33a399008c22ffefd707eefe2f57ff2ff8f1dc1929d9e3e75d3c83562d # [win64]
sha256: 06c2d7a7ef37f3a0fc124491ec9486de8f8f1fdfc25449e54b01840abbe19047 # [win64]

build:
number: 1
number: 0
skip: true # [ppc64le or not (cuda_compiler_version or "").startswith("12")]
script:
- mkdir -p $PREFIX/include # [unix]
Expand All @@ -31,28 +31,27 @@ build:

requirements:
build:
- arm-variant * {{ arm_variant_type }} # [aarch64]
- {{ compiler('c') }}
- {{ stdlib("c") }}
- {{ compiler('cxx') }}
- {{ compiler('cuda') }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to add cf-nvidia-tools to do the additional checks now? Or do we want to leave this for later?

Suggested change
- {{ compiler('cuda') }}
- {{ compiler('cuda') }}
- cf-nvidia-tools # [linux]

If we do this now, we would want to add something like this to build.sh

# Run GLIBC check
check-glibc bin/* lib/*.so.* lib64/*.so.*

This can be updated with whatever paths are appropriate to check for this library

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As this repo doesn't have build.sh and requires re-structuring (for build and test + making multiple outputs with a runtime and dev packages), I want to implement this change separately. Today I want to merge this update to get the 0.7.0 out of the door and update the feedstock later in a more time-relaxed way.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah ok. Yeah this could go in the script section above

Yep that is ok

host:
# https://github.com/NixOS/patchelf/issues/492
- patchelf !=0.18 # [linux]
# https://github.com/NixOS/patchelf/issues/492
- patchelf <0.18.0 # [linux]
run:
run_constrained:
# Only GLIBC_2.17 or older symbols present
- __glibc >=2.17 # [linux]

test:
requires:
- git
- {{ stdlib('c') }}
- {{ compiler('c') }}
- {{ compiler('cxx') }}
- {{ compiler('cuda') }}
# tests need the cuSPARSE header & libnvrtc.so
- libcusparse-dev
- cuda-nvrtc-dev
- cuda-driver-dev # [linux and (cuda_compiler_version or "").startswith("12")]
- patchelf # [linux]
files:
- test_load_elf.c # [linux]

Expand Down