-
Notifications
You must be signed in to change notification settings - Fork 751
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
Add presets for nvCOMP #1434
Add presets for nvCOMP #1434
Conversation
Looks like |
It's probably not usable on other platforms, so let's just skip it
|
Okay, sounds good. I've removed the nvCOMP installation for linux-arm64 in CI and the InfoMapper platform value for linux-arm64. |
No, I meant we can skip |
I've commented out the bitcomp headers. I had linking issues with them on Windows and then the compile issues on linux-arm64. I think perhaps we just skip bitcomp for now. |
The error on linux-arm64 is just about nvcompBitcompFormatOpts, so if you skip that it should be fine. Please try to do that instead of disabling everything. |
…InfoMapper header" This reverts commit b15ab4e.
This looks good to merge, so please revert the temporary changes to the workflow. Thanks! |
I added bindings for nvCOMP, a CUDA library for GPU accelerated lossless compression algorithms.
https://developer.nvidia.com/nvcomp
If you see the nvcomp InfoMapper, I left a TODO comment about a linking issue on Windows. Maybe you can spot something I did wrong, but I couldn't get the linking to work correctly for those symbols. Also, I will have to change the CUDA workflow back before merging, I changed it for testing the CI.