Skip to content

data flow and cmage conf#1112

Draft
dlyr wants to merge 82 commits intoSTORM-IRIT:release-candidatefrom
dlyr:rdot
Draft

data flow and cmage conf#1112
dlyr wants to merge 82 commits intoSTORM-IRIT:release-candidatefrom
dlyr:rdot

Conversation

@dlyr
Copy link
Contributor

@dlyr dlyr commented Nov 20, 2025

Pull Request Desription

Check if you branch history is PR compatible

  • Your branch need to be up to date with origin/master AND to have linear history (i.e. no merge commit).
  • Update your git repository git fetch origin if origin is this remote
  • Check with the script provided in scripts/is-history-pr-compatible.sh
  • You must use clang-format style
    These checks are enforced by github workflow actions
    Please refer to the corresponding log in case of failure

UPDATE the form below to describe your PR

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Be aware that the PR request cannot be accepted if it doesn't pass the Continuous Integration tests.

  • What kind of change does this PR introduce?

    • bug fix
    • feature
    • docs update
    • other:
  • What is the current behavior? (You can also link to an open issue here)

  • What is the new behavior (if this is a feature change)?

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

  • Other information:

@codecov
Copy link

codecov bot commented Nov 20, 2025

Codecov Report

❌ Patch coverage is 81.64794% with 49 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.19%. Comparing base (0cca427) to head (0f146f7).
⚠️ Report is 2 commits behind head on release-candidate.

Files with missing lines Patch % Lines
src/Gui/ParameterSetEditor/ParameterSetEditor.hpp 0.00% 19 Missing ⚠️
tests/unittest/tempdir.hpp 70.45% 13 Missing ⚠️
src/Dataflow/Core/Node.hpp 61.29% 12 Missing ⚠️
src/Dataflow/Core/PortIn.hpp 33.33% 2 Missing ⚠️
src/Engine/Scene/CameraManager.cpp 0.00% 2 Missing ⚠️
src/Core/Utils/StringUtils.cpp 95.65% 1 Missing ⚠️
Additional details and impacted files
@@                  Coverage Diff                  @@
##           release-candidate    #1112      +/-   ##
=====================================================
+ Coverage              41.59%   50.19%   +8.60%     
=====================================================
  Files                    387      347      -40     
  Lines                  30074    26625    -3449     
=====================================================
+ Hits                   12508    13365     +857     
+ Misses                 17566    13260    -4306     
Flag Coverage Δ
integration ?
unittests ?

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

dlyr and others added 28 commits November 25, 2025 14:48
Maybe too much ? will see when with usage.
Found by set and check...
This fixes dataflow components that are not lib, install path not
consistent.
Also try to better handle component and "_FOUND".
dlyr and others added 8 commits February 10, 2026 09:00
Since Sphinx config failed on my setup.
Works with clone in workspace, if not, filename resolution is not
working on codecov.
Hence external and build are outside of workspace, and cache must be
copied, because could not cache using ".." in path
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant