Skip to content

Conversation

@dagardner-nv
Copy link
Contributor

❄️ Code freeze for branch-25.06 and v25.06 release

What does this mean?

Only critical/hotfix level issues should be merged into branch-25.06 until release (merging of this PR).

All other development PRs should be retargeted towards the next release branch: branch-25.10.

What is the purpose of this PR?

  • Update documentation
  • Allow testing for the new release
  • Enable a means to merge branch-25.06 into main for the release

dagardner-nv and others added 11 commits January 27, 2025 09:31
….json (#530)

1. Currently, CMake adds `-L` flags to compiling object files. GCC does not warn about this, but Clang does. Suppress the noise with `-Wno-unused-command-line-argument`.

2. `CMakePresets.json` are relatively recent but allow you to easily kick off a CMake configure/build/test (or combination of all via workflow). Add an initial (opinionated) configuration that offers four options:
  * gcc-debug
  * gcc-release
  * clang-debug
  * clang-release

The advantage here is the end user doesn't have to remember a bunch of flags to pass to CMake. They can just invoke via: `cmake --workflow --preset gcc-debug`. This also offers better IDE integration, particularly in VSCode.

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

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

URL: #530
Forward-merge branch-25.02 into branch-25.06
* Replace V100 runner with an L4 runner

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

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

URL: #531
Forward-merge branch-25.02 into branch-25.06
* 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

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

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

URL: #539
* Adopt CUDA 12.8
* Remove unused download_deps script (unrelated to adopting 25.02)

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

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

URL: #536
This is #541 but having all commits rebased on `branch-25.06`

* `pybind11::error_already_set` now inherits from `std::exception` rather than `std::runtime_error`
* Adopt Pybind11 2.12, required for Python 3.12 support
* Disable Pybind's ing_ref/dec_ref checking (#542)
* Adopt updated version of versioneer, removing the need for `versioneer.py` in the repo.
* Work-around for llvm/llvm-project#92948 ref: #543
* Fetch commit history so proper version can be detected for tests

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

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

URL: #544
Upstream RAPIDS will eventually be removing boa from the default images. Since they are functionally equivalent, let's adapt.

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

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

URL: #546
* Replace the `DCHECK_LE` in the `ReusableFiberEngineFactory` class with an if/throw.
* Update `~EdgeHolder` to `LOG(DFATAL)`. The thinking here is that this error is only of use to MRC developers and not of use to downstream users (Borrowed from PR #434).

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

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

URL: #547
@dagardner-nv dagardner-nv added conda-build Enables running the conda-build step on a PR ! - Release Related to a Major Release labels Jul 14, 2025
@dagardner-nv dagardner-nv requested review from a team as code owners July 14, 2025 16:47
@dagardner-nv dagardner-nv self-assigned this Jul 14, 2025
Signed-off-by: David Gardner <dagardner@nvidia.com>
@github-project-automation github-project-automation bot moved this to Review - Approved in Morpheus Boards Jul 22, 2025
@codecov
Copy link

codecov bot commented Jul 22, 2025

Codecov Report

Attention: Patch coverage is 28.00000% with 36 lines in your changes missing coverage. Please review.

Project coverage is 73.58%. Comparing base (0a65b89) to head (84661b4).
Report is 28 commits behind head on main.

Files with missing lines Patch % Lines
cpp/mrc/include/mrc/edge/edge_holder.hpp 0.00% 12 Missing ⚠️
cpp/mrc/src/internal/system/device_info.cpp 0.00% 6 Missing ⚠️
python/mrc/_pymrc/src/executor.cpp 0.00% 5 Missing ⚠️
cpp/mrc/src/public/modules/plugins.cpp 0.00% 3 Missing ⚠️
python/mrc/_pymrc/src/operators.cpp 40.00% 3 Missing ⚠️
cpp/mrc/src/internal/runnable/engine_factory.cpp 50.00% 2 Missing ⚠️
python/mrc/_pymrc/src/segment.cpp 0.00% 2 Missing ⚠️
python/mrc/_pymrc/src/subscriber.cpp 0.00% 2 Missing ⚠️
cpp/mrc/src/public/edge/edge_builder.cpp 66.66% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #550      +/-   ##
==========================================
- Coverage   73.99%   73.58%   -0.41%     
==========================================
  Files         407      407              
  Lines       15106    15144      +38     
  Branches     1198     1204       +6     
==========================================
- Hits        11177    11144      -33     
- Misses       3929     4000      +71     
Flag Coverage Δ
cpp 70.52% <20.00%> (+1.08%) ⬆️
py 43.54% <14.00%> (-0.54%) ⬇️

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

Files with missing lines Coverage Δ
cpp/mrc/src/internal/system/topology.cpp 83.85% <ø> (ø)
cpp/mrc/src/tests/test_topology.cpp 90.76% <ø> (-7.65%) ⬇️
python/mrc/_pymrc/include/pymrc/coro.hpp 91.17% <100.00%> (-2.95%) ⬇️
python/mrc/_pymrc/src/module_wrappers/pickle.cpp 100.00% <100.00%> (ø)
...n/mrc/_pymrc/src/module_wrappers/shared_memory.cpp 84.31% <100.00%> (+0.98%) ⬆️
python/mrc/_pymrc/src/utilities/deserializers.cpp 78.57% <100.00%> (ø)
cpp/mrc/src/public/edge/edge_builder.cpp 55.93% <66.66%> (ø)
cpp/mrc/src/internal/runnable/engine_factory.cpp 83.95% <50.00%> (-1.77%) ⬇️
python/mrc/_pymrc/src/segment.cpp 77.72% <0.00%> (-3.11%) ⬇️
python/mrc/_pymrc/src/subscriber.cpp 27.27% <0.00%> (ø)
... and 5 more

... and 8 files 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 0a65b89...84661b4. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

* Python tests depend on `pytest-asyncio`
* Fix local CI bug

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

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

URL: #552
@dagardner-nv dagardner-nv merged commit f0cd754 into main Jul 23, 2025
39 checks passed
@github-project-automation github-project-automation bot moved this from Review - Approved to Done in Morpheus Boards Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

! - Release Related to a Major Release conda-build Enables running the conda-build step on a PR

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants