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

Disable shared memory by default, and fix --stop_after flag for vdb_upload example #1755

Merged

Conversation

dagardner-nv
Copy link
Contributor

Description

  • Disable shared memory by default since the C++ impl of the Triton stage doesn't support it
  • Set the default source type to rss since it is the default, but defaulting to None causes any RSS related CLI flags to be ignored

Closes #1753

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 dagardner-nv self-assigned this Jun 24, 2024
@dagardner-nv dagardner-nv requested a review from a team as a code owner June 24, 2024 18:31
@dagardner-nv dagardner-nv added bug Something isn't working non-breaking Non-breaking change labels Jun 24, 2024
@dagardner-nv
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 039bcf2 into nv-morpheus:branch-24.06 Jun 24, 2024
15 checks passed
@dagardner-nv dagardner-nv deleted the david-vdb-upload-2406 branch June 24, 2024 22:41
rapids-bot bot pushed a commit that referenced this pull request Jun 24, 2024
…1758)

* PR #1617 replaced the hard-coded questions with a command-line-flag, however it was not given a default value nor was it marked as a required flag.
* Set the original question as the default value for the flag

Requires #1755
Closes #1757


## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/nv-morpheus/Morpheus/blob/main/docs/source/developer_guide/contributing.md).
- 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.

Authors:
  - David Gardner (https://github.com/dagardner-nv)

Approvers:
  - Michael Demoret (https://github.com/mdemoret-nv)

URL: #1758
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working non-breaking Non-breaking change
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[BUG]: vdb_upload should not use shared memory
2 participants