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

Build: Replace NGC ZIP archive HTTPS downloads with NGC CLI #657

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

tbirdso
Copy link
Contributor

@tbirdso tbirdso commented Jan 16, 2025

Update NGC data retrieval to address NGC ZIP HTTPS download support deprecation.

As of 2025-01-15 NVIDIA NGC has fully deprecated support for retrieving resources as a ZIP archive via HTTPS tools (curl, wget). This commit updates project CMake calls to holoscan_download_data to remove the https prefix and prompt the Holoscan SDK download_ngc_data script to use the NGC CLI rather than curl or wget to retrieve sample data. MD5 hashsum checks are not supported for NGC CLI downloads with the download_ngc_data script as of Holoscan SDK v2.8.0.

Developers not developing in the HoloHub container should run ./run setup on their system to install HoloHub prerequisites including the NGC CLI tool.

The Holoscan SDK NGC download script is located at: https://raw.githubusercontent.com/nvidia-holoscan/holoscan-sdk/refs/tags/v2.8.0/scripts/download_ngc_data

Affected URLs were identified and replaced with VS Code regex matching:

nvidia-holoscan/holoscan-sdk#42

Update NGC data retrieval to address NGC ZIP HTTPS download support deprecation.

As of 2025-01-15 NVIDIA NGC has fully deprecated support for retrieving
resources as a ZIP archive via HTTPS tools (curl, wget). This commit
updates project CMake calls to `holoscan_download_data` to remove the
`https` prefix and prompt the Holoscan SDK `download_ngc_data` script to
use the NGC CLI rather than curl or wget to retrieve sample data. MD5
hashsum checks are not supported for NGC CLI downloads with the
`download_ngc_data` script as of Holoscan SDK v2.8.0.

Developers not developing in the HoloHub container should run `./run
setup` on their system to install HoloHub prerequisites including the
NGC CLI tool.

The Holoscan SDK NGC download script is located at:
https://raw.githubusercontent.com/nvidia-holoscan/holoscan-sdk/refs/tags/v2.8.0/scripts/download_ngc_data

Affected URLs were identified and replaced with VS Code regex matching:
- Find: https://api.ngc.nvidia.com/v2/resources/([a-zA-Z0-9/_-]+)/versions/([0-9a-zA-Z.]+)/zip
- Replace: $1:$2

nvidia-holoscan/holoscan-sdk#42
Signed-off-by: Tom Birdsong <tbirdsong@nvidia.com>
Copy link
Contributor

@wendell-hom wendell-hom left a comment

Choose a reason for hiding this comment

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

Looks good, thanks Tom!

@tbirdso tbirdso merged commit 4e1da8a into nvidia-holoscan:main Jan 16, 2025
3 checks passed
tbirdso added a commit that referenced this pull request Jan 16, 2025
Update NGC data retrieval to address NGC ZIP HTTPS download support deprecation.

As of 2025-01-15 NVIDIA NGC has fully deprecated support for retrieving
resources as a ZIP archive via HTTPS tools (curl, wget). This commit
updates project CMake calls to `holoscan_download_data` to remove the
`https` prefix and prompt the Holoscan SDK `download_ngc_data` script to
use the NGC CLI rather than curl or wget to retrieve sample data. MD5
hashsum checks are not supported for NGC CLI downloads with the
`download_ngc_data` script as of Holoscan SDK v2.8.0.

Developers not developing in the HoloHub container should run `./run
setup` on their system to install HoloHub prerequisites including the
NGC CLI tool.

The Holoscan SDK NGC download script is located at:
https://raw.githubusercontent.com/nvidia-holoscan/holoscan-sdk/refs/tags/v2.8.0/scripts/download_ngc_data

Affected URLs were identified and replaced with VS Code regex matching:
- Find: https://api.ngc.nvidia.com/v2/resources/([a-zA-Z0-9/_-]+)/versions/([0-9a-zA-Z.]+)/zip
- Replace: $1:$2

nvidia-holoscan/holoscan-sdk#42

Signed-off-by: Tom Birdsong <tbirdsong@nvidia.com>
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.

2 participants