Skip to content

[Core] Move SceneCheckerVisitor out of the SceneChecking project into the core#6150

Open
bakpaul wants to merge 5 commits into
sofa-framework:masterfrom
bakpaul:26_06_enable_checking_in_python
Open

[Core] Move SceneCheckerVisitor out of the SceneChecking project into the core#6150
bakpaul wants to merge 5 commits into
sofa-framework:masterfrom
bakpaul:26_06_enable_checking_in_python

Conversation

@bakpaul

@bakpaul bakpaul commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Moves the SceneCheck visitor into the core. This is required for the SofaPython3 PR enabling scene checking when launching a scene using python.

[with-all-tests]
[force-full-build]


By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).


Reviewers will merge this pull-request only if

  • it builds with SUCCESS for all platforms on the CI.
  • it does not generate new warnings.
  • it does not generate new unit test failures.
  • it does not generate new scene test failures.
  • it does not break API compatibility.
  • it is more than 1 week old (or has fast-merge label).

@bakpaul bakpaul added pr: fast merge Minor change that can be merged without waiting for the 7 review days pr: status to review To notify reviewers to review this pull-request labels Jun 11, 2026
@sofabot

sofabot commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

[ci-depends-on] detected during build #1.

To unlock the merge button, you must

@hugtalbot hugtalbot added this to the v26.06 milestone Jun 12, 2026
@sofabot

sofabot commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

[ci-depends-on] detected during build #2.

To unlock the merge button, you must

@fredroy fredroy force-pushed the 26_06_enable_checking_in_python branch from 44ffc7d to 314e8fb Compare June 15, 2026 03:50
@sofabot

sofabot commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

[ci-depends-on] detected during build #3.

To unlock the merge button, you must

@fredroy fredroy added the pr: breaking Change possibly inducing a compilation error label Jun 15, 2026
@fredroy

fredroy commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

no compatibility ? 🫢
(the headers moved and the namespace changed...)

Comment thread Sofa/framework/Simulation/Core/src/sofa/simulation/SceneCheckerVisitor.cpp Outdated
@hugtalbot

Copy link
Copy Markdown
Contributor

Agreed with @fredroy
Ready after compat'

@bakpaul

bakpaul commented Jun 15, 2026

Copy link
Copy Markdown
Contributor Author

You think that anyone implemented a scene check on his own outside of the core devs ? And even if it is the case, this is the visitor, surely no one is using it outside thee core dev. I really think the "compatibility layer" should be restricted to parts of the code that really has potential to be used by the community, because it takes time to take care of it properly. But well if necessary then I'll do it.

@fredroy

fredroy commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

You think that anyone implemented a scene check on his own outside of the core devs ? And even if it is the case, this is the visitor, surely no one is using it outside thee core dev. I really think the "compatibility layer" should be restricted to parts of the code that really has potential to be used by the community, because it takes time to take care of it properly. But well if necessary then I'll do it.

Well, the problem is how to judge something is "really has potential to be used by the community,".
But personally I am no hell-bent on this compat (I did not set "Request changes" 🫢), but it needs at least the flag pr:breaking , to be notified by the changelog produced at the release stage.

@sofabot

sofabot commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

[ci-depends-on] detected during build #4.

To unlock the merge button, you must

@fredroy fredroy force-pushed the 26_06_enable_checking_in_python branch from 4f1e8ec to f58d084 Compare June 16, 2026 22:18
@sofabot

sofabot commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

[ci-depends-on] detected during build #5.

To unlock the merge button, you must

@fredroy

fredroy commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

[ci-build][with-all-tests]

@sofabot

sofabot commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

[ci-depends-on] detected during build #6.

To unlock the merge button, you must

@sofabot

sofabot commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

[ci-depends-on] detected during build #7.

To unlock the merge button, you must

@bakpaul

bakpaul commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

[ci-build][with-all-tests]

@sofabot

sofabot commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

[ci-depends-on] detected during build #8.

To unlock the merge button, you must

@fredroy

fredroy commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

[ci-build][with-all-tests]

@sofabot

sofabot commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

[ci-depends-on] detected during build #9.

To unlock the merge button, you must

@hugtalbot

Copy link
Copy Markdown
Contributor

Compilation fails :

external_directories/fetched/SofaPython3/bindings/Sofa/src/SofaPython3/Sofa/Simulation/Submodule_Simulation.cpp:74:92: error: 'PluginLoadStatus' has not been declared
   74 |         if(res == sofa::helper::system::PluginManager::PluginLoadStatus::SUCCESS || res == PluginLoadStatus::ALREADY_LOADED)

@sofa-framework sofa-framework deleted a comment from hugtalbot Jun 18, 2026
@fredroy fredroy force-pushed the 26_06_enable_checking_in_python branch from 1bedcb9 to 25f28c9 Compare June 18, 2026 06:56
@sofabot

sofabot commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

[ci-depends-on] detected during build #10.

To unlock the merge button, you must

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: breaking Change possibly inducing a compilation error pr: fast merge Minor change that can be merged without waiting for the 7 review days pr: status to review To notify reviewers to review this pull-request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants