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

Document each of the Conda environment files #1932

Merged
merged 410 commits into from
Oct 22, 2024
Merged

Document each of the Conda environment files #1932

merged 410 commits into from
Oct 22, 2024

Conversation

dagardner-nv
Copy link
Contributor

Description

Closes #1660

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
Switch to using the array_pkg fixture
e586501
@dagardner-nv
Lint fixes
f859ca0
@dagardner-nv
Add a few vocabulary entries
bc6ac3d
@dagardner-nv
WIP
e898922
@dagardner-nv
WIP
16930df
@dagardner-nv
Passthru stage should support GPU & CPU modes
68c5f83
@dagardner-nv
First pass at updating AppShieldSourceStage to emit ControlMessages
4f92ccb
@dagardner-nv
update tests
1bc26a0
@dagardner-nv
WIP
0b4b7ba
@dagardner-nv
Set the http port to be the default since we are running in C++ mode
76c16ce
@dagardner-nv
Fix spelling error
a16bc90
@dagardner-nv
Remove final decorator as this conflicts with using the preallocator …
be15217 
…mixin
@dagardner-nv
Deprecate AppShieldMessageMeta
36b0190
@dagardner-nv
Explicitly convert source dataframes to cudf
5bd82f2
@dagardner-nv
Refactor to use control messages, and limit pandas<->cudf conversions
9835e5f
@dagardner-nv
wip
2ad1c99
@dagardner-nv
Use PreallocatorMixin in create features since its emitting new dataf…
8d2f424 
…rames
@dagardner-nv
Use C++ mode
206bc4e
@dagardner-nv
Relocate test and update
45291ea
@dagardner-nv
Update tests
cf4aa84
@dagardner-nv
Config.freeze enforces a compatible value for CppConfig on the first …
fc9e66b 
…call, raises a ValueError on mismatch on subsequent calls
@dagardner-nv
Ensure the empty config created for boundary stages have a matching e…
f6d5513 
…xecution mode
@dagardner-nv
Boundary stages should support CPU mode
45dc802
@dagardner-nv
Remove setting CppConfig as the Config class does this now
ca81a8d
@dagardner-nv
Add type hints
89e2add
@dagardner-nv
First pass at deprecating UserMessageMeta and updating AutoencoderSou…
51b4833 
…rceStage to emit ControlMessages
@dagardner-nv
AppShieldSourceStage should support both cpu and gpu modes
3370623
@dagardner-nv
Cleanup comments
8baf123
@dagardner-nv
Update TrainAE to accept incoming ControlMessages
8bf6bc4
@dagardner-nv
Merge branch 'branch-24.10' of github.com:nv-morpheus/Morpheus into d…
58ab86e 
…avid-cpu-only-mode-1846
@dagardner-nv
Fix imports
7be0402
@dagardner-nv
Rase a ValueError instead of a KeyError to match behavior of the C++ …
fa67db6 
…impl
@dagardner-nv
Remove inconsistent tensor reshaping issue nv-morpheus#1955
485764c
@dagardner-nv
Update ControlMemory tests to test the Python and C++ impls
d903feb
@dagardner-nv
Test should be run in both cpu and gpu modes
ef466ae
@dagardner-nv
Remove unused config_only_cpp and config_no_cpp fixtures
357a6e2
@dagardner-nv
lint fixes
1998922
@dagardner-nv
Fix cpu-mode json tests
a0f5e5b
@dagardner-nv
Fix misleading runtime erro messafe, remove unused inferring gpu mode…
80e817a 
… based on the use_cudf marker
@dagardner-nv
Ignore code.visualstudio.com, these urls are being forbidden when run…
32fb3b8 
… from inside CI
@dagardner-nv
Merge branch 'david-cpu-only-mode-1846' into david-cpu-only-llm
13a0eb6
@dagardner-nv
Merge branch 'david-cpu-only-llm' into david-cpu-only-docs
acff1f0
@dagardner-nv
Improved doc strings
341ffe0
@dagardner-nv
Merge branch 'branch-24.10' of github.com:nv-morpheus/Morpheus into d…
fe58767 
…avid-cpu-only-llm
@dagardner-nv
Merge branch 'david-cpu-only-llm' into david-cpu-only-docs
cabfc51
@dagardner-nv
Misc docstring improvements
368ea5d
@dagardner-nv
Merge branch 'branch-24.10' of github.com:nv-morpheus/Morpheus into d…
77c5afc 
…avid-cpu-only-docs
@mdemoret-nv
Copy link
Contributor

@dagardner-nv Need to update this to remove diffs

@dagardner-nv
Copy link
Contributor Author

/ok to test

mdemoret-nv
mdemoret-nv approved these changes Oct 21, 2024
View reviewed changes
docs/source/developer_guide/contributing.md Outdated Show resolved Hide resolved
docs/source/developer_guide/contributing.md Outdated Show resolved Hide resolved
@dagardner-nv
Copy link
Contributor Author

/ok to test

@dagardner-nv
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 674e629 into nv-morpheus:branch-24.10 Oct 22, 2024
10 of 11 checks passed
@dagardner-nv dagardner-nv deleted the david-contrib-conda-yaml-1660 branch October 22, 2024 01:10
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

@dagardner-nv dagardner-nv

Labels
doc Improvements or additions to documentation non-breaking Non-breaking change
Projects
Morpheus Boards
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[DOC]: Developer and Contributor guide should explain the conda yaml files and when to use them
3 participants
@dagardner-nv @mdemoret-nv @yczhang-nv