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

Use ${PROJECT_NAME}-exports not ${ROJECT_NAME}-core-exports for dependencies in CMake scripts #2155

Draft
wants to merge 9 commits into
base: branch-25.06
Choose a base branch
from

Conversation

dagardner-nv
Copy link
Contributor

Description

Closes #2149

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 and others added 8 commits January 28, 2025 16:22
… on shutdown (nv-morpheus#2137)

* Don't log error messages if `_server_close_event` has already been set.
* Unrelated documentation improvement for the RabbitMQ examples to ensue the `MORPHEUS_ROOT` environment variable is set.

Closes nv-morpheus#2135

## 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:
  - Eli Fajardo (https://github.com/efajardo-nv)

URL: nv-morpheus#2137
- Update reference to output file to match name used in command
- Change `0` and `1` to `false` and `true` in example results to match what was seen after running example

## 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:
  - Eli Fajardo (https://github.com/efajardo-nv)

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

URL: nv-morpheus#2138
… function calls (nv-morpheus#2140)

<img width="532" alt="image" src="https://github.com/user-attachments/assets/39f491c6-4551-44a1-94a9-d3f420cf78e1" />

- [nv-morpheus#2121](nv-morpheus#2121) ensures the cursor is reset to the last line of output after shutdown by adding `ProgressBarContextManager::mark_pbar_as_completed()`. 
- When executing some of the pipelines, race conditions may cause calling `ProgressBarContextManager::display_all()` after `ProgressBarContextManager::mark_pbar_as_completed()`, which intermittently causes the log display issue above - which should has been fixed by this PR.

Closes nv-morpheus#2119 

## 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:
  - Yuchen Zhang (https://github.com/yczhang-nv)
  - David Gardner (https://github.com/dagardner-nv)

Approvers:
  - David Gardner (https://github.com/dagardner-nv)
  - Will Killian (https://github.com/willkill07)

URL: nv-morpheus#2140
@dagardner-nv dagardner-nv added bug Something isn't working non-breaking Non-breaking change DO NOT MERGE PR should not be merged; see PR for details skip-ci Optionally Skip CI for this PR labels Jan 29, 2025
@dagardner-nv dagardner-nv self-assigned this Jan 29, 2025
@dagardner-nv dagardner-nv requested review from a team as code owners January 29, 2025 22:07
@dagardner-nv dagardner-nv marked this pull request as draft January 29, 2025 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working DO NOT MERGE PR should not be merged; see PR for details non-breaking Non-breaking change skip-ci Optionally Skip CI for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG]: examples/developer_guide/3_simple_cpp_stage fails to build in release container
3 participants