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.

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

Include requirements files in the morpheus packages #1957

Merged
merged 6 commits into from
Oct 19, 2024
Merged

Include requirements files in the morpheus packages #1957

merged 6 commits into from
Oct 19, 2024

Conversation

AnuradhaKaruppiah
Copy link
Contributor

@AnuradhaKaruppiah AnuradhaKaruppiah commented Oct 18, 2024 •
edited
Loading

pip dependencies are included as a part of the package -
morpheus/requirements_morpheus_core.txt
morpheus_dfp/requirements_morpheus_dfp.txt
morpheus_llm/requirements_morpheus_llm.txt

And can be installed as needed. For example -

import importlib.resources
import subprocess
requirements_file = importlib.resources.path("morpheus_llm", "requirements_morpheus_llm.txt")
subprocess.call(f"pip install -r {requirements_file}".split())

@copy-pr-bot copy-pr-bot
Copy link

copy-pr-bot bot commented Oct 18, 2024

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.

@AnuradhaKaruppiah AnuradhaKaruppiah self-assigned this Oct 18, 2024
@AnuradhaKaruppiah AnuradhaKaruppiah added non-breaking Non-breaking change feature request New feature or request conda-build Enables running the conda-build step on a PR labels Oct 18, 2024
@AnuradhaKaruppiah
Copy link
Contributor Author

/ok to test

@AnuradhaKaruppiah
Copy link
Contributor Author

/ok to test

@AnuradhaKaruppiah AnuradhaKaruppiah changed the title Include requirements file in the morpheus_llm package Include requirements files in the morpheus package Oct 18, 2024
@AnuradhaKaruppiah AnuradhaKaruppiah changed the title Include requirements files in the morpheus package Include requirements files in the morpheus packages Oct 18, 2024
@AnuradhaKaruppiah
Copy link
Contributor Author

/ok to test

@AnuradhaKaruppiah
Copy link
Contributor Author

/ok to test

@AnuradhaKaruppiah
Copy link
Contributor Author

/ok to test

@AnuradhaKaruppiah
Copy link
Contributor Author

/ok to test

@AnuradhaKaruppiah
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit e648a5b into nv-morpheus:branch-24.10 Oct 19, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mdemoret-nv mdemoret-nv mdemoret-nv approved these changes

Assignees

@AnuradhaKaruppiah AnuradhaKaruppiah

Labels
conda-build Enables running the conda-build step on a PR feature request New feature or request non-breaking Non-breaking change
Projects
Morpheus Boards
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AnuradhaKaruppiah @mdemoret-nv