Skip to content

[WIP] Feature/nvimagecodec source and tests #980

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 70 commits into from
Closed

[WIP] Feature/nvimagecodec source and tests #980

wants to merge 70 commits into from

Conversation

@cdinea
Copy link

@cdinea cdinea commented Nov 21, 2025

No description provided.

cdinea and others added 30 commits November 19, 2025 16:01
@cdinea @gforsyth
refactor: Improve nvImageCodec integration and conda configuration
36e975c
@cdinea @gforsyth
feat: Consolidate nvImageCodec dependencies in dependencies.yaml
b75ba5f
@cdinea @gforsyth
build: Update CMake and build configurations for cuslide2
fc1986b 
- Fix libjpeg-turbo cmake configuration for both cuslide and cuslide2
- Update nvimgcodec cmake dependency configuration
- Update examples CMakeLists
- Update build scripts and documentation
@cdinea @gforsyth
chore: Add PR_DESCRIPTION.md to .gitignore
4c688d8
@cdinea @gforsyth
feat: Add comprehensive nvImageCodec installation verification script
d3e322e
@cdinea @gforsyth
refactor: Remove nvImageCodec plugin implementation references from C…
13670f7 
…MakeLists
@cdinea @gforsyth
Add CMake utility modules for cuslide2 plugin
678c40a
@cdinea @gforsyth
fix: Comment out cuslide2_nvimagecodec_test example for infrastructur…
fec4ee5 
…e-only PR
@cdinea @gforsyth
build: Add missing CMake dependency files for cuslide2 plugin
e164af8
@cdinea @gforsyth
revert: Remove cuslide2 example from general examples directory
f737c12
@cdinea @gforsyth
revert: Restore original notebook README and clean .gitignore
67a37fe
@cdinea @gforsyth
fix: Disable tests/benchmarks and add missing config file for cuslide2
4486620
@cdinea @gforsyth
Add libnvimgcodec dependencies to dependencies.yaml and regenerate en…
b439403 
…vironment files
@cdinea @gforsyth
Use yasm instead of nasm for libjpeg-turbo assembly compilatio
68fc1e2
@cdinea @gforsyth
Add nvImageCodec installation verification script
119de8c
@cdinea @gforsyth
Update pyproject.toml with nvimgcodec dependencies and remove old scr…
405c5fd 
…ipt location
@cdinea @gforsyth
Add comprehensive cuslide2 infrastructure verification script
28a2ef9
@cdinea @gforsyth
Refactor verification script and fix build configuration
b0d9634
@cdinea @gforsyth
Add nvImageCodec infrastructure setup for cuslide2
ad0d955
@cdinea @gforsyth
style: Apply SPDX license headers and formatting fixes
60385e0
@cdinea @gforsyth
fixed with ruff per Greg
5f8a8cd 
Signed-off-by: cdinea <[email protected]>
@cdinea @gforsyth
added comment detail
543d629 
Signed-off-by: cdinea <[email protected]>
@cdinea @gforsyth
fix: Update copyright headers to SPDX format and fix typo
17148e2 
Signed-off-by: cdinea <[email protected]>
@cdinea @gforsyth
added comment detail
4514538 
Signed-off-by: cdinea <[email protected]>
@cdinea @jakirkham @gforsyth
Update dependencies.yaml
8c594d6 
Co-authored-by: jakirkham <[email protected]>
@cdinea @jakirkham @gforsyth
Update dependencies.yaml per john's suuggestion
dd7d107 
Co-authored-by: jakirkham <[email protected]>
@cdinea @jakirkham @gforsyth
Update dependencies.yaml per john's suggestion
85dce8e 
Co-authored-by: jakirkham <[email protected]>
@cdinea @jakirkham @gforsyth
Update dependencies.yaml per john's suggestion
6076d49 
Co-authored-by: jakirkham <[email protected]>
@cdinea @gforsyth
build: Regenerate conda environment files from dependencies.yaml
7bd8d9b 
Signed-off-by: cdinea <[email protected]>
@cdinea @gforsyth
build: Fix spacing in pyproject.toml nvidia-nvimgcodec dependency
27acb53 
Signed-off-by: cdinea <[email protected]>
AyodeAwe and others added 17 commits November 19, 2025 13:21
@AyodeAwe @cdinea
Revert "Forward-merge release/25.12 into main" (rapidsai#973)
51345ca 
Reverts rapidsai#972
@cdinea
chore: remove unnecesary files
55b0eb5
@cdinea
chore: remove development scripts, logs, and IDE configuration files
bbaa425
@cdinea
chore: remove backup file
bfa81f7
@cdinea
chore: remove incomplete header-only test file
5166b8e
@cdinea
chore: remove duplicate nvImageCodec files from decoding/ directory
bd25cba
@cdinea
perf: wrap debug fmt::print statements with #ifdef DEBUG guards
67ad1dc
@cdinea
Wrap ThreadBatchDataLoader debug prints in #ifdef DEBUG guards
1f02fa9
@cdinea
refactor: remove redundant empty() check in JPEG data validation
a82c46a
@cdinea
chore: remove unused files and includes
34eb295
@cdinea
fix: implement proper cleanup in NvImageCodecManager destructor
83c4659
@cdinea
fix: Add CPU decoder stubs include and fix unused variable warning
6c1d6f4
@cdinea
fix: Correct CUDA memory allocation in read_region and add test script
d520c99
@cdinea
fix: Add Philips TIFF metadata detection workaround and test script
6709ede
@cdinea
Fix Philips TIFF and Aperio SVS support
0f39759
@cdinea
Fix segfault in old cuslide plugin: use cudaMemcpy for GPU destinations
d173ee9
@cdinea
refactor: Clean up nvImageCodec decoder and parser separation of conc…
17e6bb5 
…erns
@cdinea cdinea requested review from a team as code owners November 21, 2025 21:57
@cdinea cdinea requested a review from gforsyth November 21, 2025 21:57
@copy-pr-bot
Copy link

copy-pr-bot bot commented Nov 21, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@cdinea cdinea changed the title Feature/nvimagecodec source and tests [WIP] Feature/nvimagecodec source and tests Nov 21, 2025
@cdinea cdinea added the improvement Improves an existing functionality label Nov 22, 2025
@cdinea cdinea added this to the v25.12.00 milestone Nov 22, 2025
@cdinea cdinea force-pushed the feature/nvimagecodec-source-and-tests branch 2 times, most recently from 102cc64 to 17e6bb5 Compare November 24, 2025 19:29
@cdinea cdinea requested a review from a team as a code owner November 24, 2025 19:29
@cdinea cdinea closed this Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gforsyth gforsyth Awaiting requested review from gforsyth gforsyth is a code owner automatically assigned from rapidsai/packaging-codeowners

Assignees

No one assigned

Labels

improvement Improves an existing functionality

Projects

None yet

Milestone

v25.12.00

Development

Successfully merging this pull request may close these issues.

3 participants

@cdinea @gforsyth @AyodeAwe