Skip to content

Support running the DLP example within the release container #2244

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
Merged
rapids-bot merged 13 commits into from
Jul 11, 2025
Merged

Support running the DLP example within the release container #2244

rapids-bot merged 13 commits into from
Jul 11, 2025

Conversation

@dagardner-nv
Copy link
Contributor

@dagardner-nv dagardner-nv commented Jul 11, 2025

Description

  • Support the DLP pipeline in the release container
  • Add a compile.sh script allowing the example to be built after the fact.
  • Since we don't include models in the release container (nor do we provide a way to fetch them), this also includes a fallback mechanism for fetching the model on demand from huggingface hub.

Closes #2242

By Submitting this PR I confirm:

  • I am familiar with the Contributing Guidelines.
  • When the PR is ready for review, new or existing tests cover these changes.
  • When the PR is ready for review, the documentation is up to date with these changes.

@dagardner-nv
Fallback to loading the model from huggingface hub
3cd4114 
Signed-off-by: David Gardner <[email protected]>
@dagardner-nv
Restructure code to fallback to downloading the model from huggingfac…
3ea52d7 
…e hub if the model doesn't exist locally

Signed-off-by: David Gardner <[email protected]>
@dagardner-nv
Remove unused imports
9320685 
Signed-off-by: David Gardner <[email protected]>
@dagardner-nv
Add compile script to allow building in the release container
666fd06 
Signed-off-by: David Gardner <[email protected]>
@dagardner-nv
WIP
5dd0998 
Signed-off-by: David Gardner <[email protected]>
@dagardner-nv
Don't require the labels file to exist
0c538b4 
Signed-off-by: David Gardner <[email protected]>
@dagardner-nv
Remove the warning log
718abab 
Signed-off-by: David Gardner <[email protected]>
@dagardner-nv
Update instructions
6bf3ca4 
Signed-off-by: David Gardner <[email protected]>
@dagardner-nv
Remove the need to cd into the directory
5ea0b3c 
Signed-off-by: David Gardner <[email protected]>
@dagardner-nv
Remove uneeded files
7a71dbb 
Signed-off-by: David Gardner <[email protected]>
@dagardner-nv
Document that this is working now
19cbf1f 
Signed-off-by: David Gardner <[email protected]>
@dagardner-nv dagardner-nv self-assigned this Jul 11, 2025
@dagardner-nv dagardner-nv requested a review from a team as a code owner July 11, 2025 18:59
@dagardner-nv dagardner-nv added bug Something isn't working non-breaking Non-breaking change labels Jul 11, 2025
@morpheus-bot-test morpheus-bot-test bot moved this to Review - Ready for Review in Morpheus Boards Jul 11, 2025
@dagardner-nv
Fix CR year
2c8443a 
Signed-off-by: David Gardner <[email protected]>
@dagardner-nv dagardner-nv changed the title David dlp release container Support running the DLP example within the release container Jul 11, 2025
@github-project-automation github-project-automation bot moved this from Review - Ready for Review to Review - Approved in Morpheus Boards Jul 11, 2025
@dagardner-nv
Copy link
Contributor Author

dagardner-nv commented Jul 11, 2025

/merge

@rapids-bot rapids-bot bot merged commit 77e6dd2 into nv-morpheus:branch-25.06 Jul 11, 2025
14 checks passed
@github-project-automation github-project-automation bot moved this from Review - Approved to Done in Morpheus Boards Jul 11, 2025
@dagardner-nv dagardner-nv deleted the david-dlp-release-container branch July 11, 2025 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@willkill07 willkill07 willkill07 approved these changes

Assignees

@dagardner-nv dagardner-nv

Labels

bug Something isn't working non-breaking Non-breaking change

Projects

Morpheus Boards
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FEA]: Support running the DLP example from the release container

2 participants

@dagardner-nv @willkill07