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

Bugfix: Remove MRC_USE_CONDA option since it must always be On #539

Conversation

willkill07
Copy link
Member

@willkill07 willkill07 commented Feb 5, 2025

Description

  • Removes MRC_USE_CONDA since it must always be on.
  • Cleans up some CMake + build files along the way.
  • Fixes some minor linting issues in mrc.code-workspace

This necessary as part of nv-morpheus/Morpheus#2154

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@willkill07 willkill07 requested a review from a team as a code owner February 5, 2025 16:22
@willkill07 willkill07 added non-breaking Non-breaking change bug Something isn't working breaking Breaking change and removed non-breaking Non-breaking change labels Feb 5, 2025
Copy link
Contributor

@dagardner-nv dagardner-nv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you need to update scripts/compile.sh as well, and a few more in the ./ci dir

@willkill07
Copy link
Member Author

@dagardner-nv this is MRC, not Morpheus

@dagardner-nv dagardner-nv self-requested a review February 5, 2025 16:41
rapids-bot bot pushed a commit to nv-morpheus/utilities that referenced this pull request Feb 5, 2025
#99)

Yet another PR for the cleanup of `_USE_CONDA` options

This is another step in nv-morpheus/Morpheus#2154 which is required by Morpheus after merging nv-morpheus/MRC#539

Authors:
  - Will Killian (https://github.com/willkill07)

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

URL: #99
@willkill07
Copy link
Member Author

/merge

@rapids-bot rapids-bot bot merged commit 5d354df into nv-morpheus:branch-25.06 Feb 5, 2025
22 checks passed
Copy link

codecov bot commented Feb 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.03%. Comparing base (a4ad85a) to head (6d19441).
Report is 1 commits behind head on branch-25.06.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff                @@
##           branch-25.06     #539      +/-   ##
================================================
+ Coverage         73.98%   74.03%   +0.05%     
================================================
  Files               407      407              
  Lines             15104    15103       -1     
  Branches           1198     1198              
================================================
+ Hits              11174    11181       +7     
+ Misses             3930     3922       -8     
Flag Coverage Δ
cpp 70.68% <ø> (+0.05%) ⬆️
py 44.07% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a4ad85a...6d19441. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Breaking change bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants