Skip to content

Conversation

weiji14
Copy link
Member

@weiji14 weiji14 commented Jul 24, 2025

Available since conda-forge/nvcomp-feedstock#23

The conda-forge package actually reuses the same nvidia-nvcomp-cu12==4.2.0.14 wheel from PyPI (see https://github.com/conda-forge/nvcomp-feedstock/blob/e255600ba4eeec9fcb966b03b9c1cd76e5952af4/recipe/meta.yaml#L11-L12), so should be practically the same.

Have tested running benchmarks/zstd_benchmark.py locally, and seems to work fine:

$ python benchmarks/zstd_benchmark.py
Opened array with compressors: (ZstdCodec(level=0, checksum=False),)
Total time to read data: 1.8253955841064453 s
Effective I/O bandwidth: 0.534986776840506 GB/s
Opened array with compressors: (NvcompZstdCodec(level=0, checksum=False),)
Total time to read data: 1.3693149089813232 s
Effective I/O bandwidth: 0.7131759784361771 GB/s

Comment on lines +6034 to +6043
version: 3.0.5.dev161+gf89b2324
manager: pip
platform: linux-64
dependencies:
donfig: '>=0.8'
numcodecs: '>=0.14'
numpy: '>=1.25'
numpy: '>=1.26'
packaging: '>=22.0'
typing-extensions: '>=4.9'
url: git+https://github.com/akshaysubr/zarr-python.git@a8c0db34ef488d654bac92ee5805bbffd2d7d8dc
url: git+https://github.com/akshaysubr/zarr-python.git@f89b232486d040047364acfbb6d3825b8b6a2a6b
Copy link
Member Author

@weiji14 weiji14 Jul 24, 2025

Choose a reason for hiding this comment

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

zarr-developers/zarr-python#2863 should be close to merging, so can also wait a few days/weeks to remove this last pip-installed package from the environment.yml.

Copy link
Member Author

Choose a reason for hiding this comment

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

That PR is taking longer than I thought 😅, so I'll just merge this first.

@weiji14 weiji14 merged commit 716d38a into main Sep 1, 2025
@weiji14 weiji14 deleted the conda-forge/nvcomp branch September 1, 2025 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant