[Core] Move SceneCheckerVisitor out of the SceneChecking project into the core#6150
[Core] Move SceneCheckerVisitor out of the SceneChecking project into the core#6150bakpaul wants to merge 5 commits into
Conversation
|
[ci-depends-on] detected during build #1. To unlock the merge button, you must
|
|
[ci-depends-on] detected during build #2. To unlock the merge button, you must
|
44ffc7d to
314e8fb
Compare
|
[ci-depends-on] detected during build #3. To unlock the merge button, you must
|
|
no compatibility ? 🫢 |
|
Agreed with @fredroy |
|
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,". |
|
[ci-depends-on] detected during build #4. To unlock the merge button, you must
|
4f1e8ec to
f58d084
Compare
|
[ci-depends-on] detected during build #5. To unlock the merge button, you must
|
|
[ci-build][with-all-tests] |
|
[ci-depends-on] detected during build #6. To unlock the merge button, you must
|
|
[ci-depends-on] detected during build #7. To unlock the merge button, you must
|
|
[ci-build][with-all-tests] |
|
[ci-depends-on] detected during build #8. To unlock the merge button, you must
|
|
[ci-build][with-all-tests] |
|
[ci-depends-on] detected during build #9. To unlock the merge button, you must
|
|
Compilation fails : |
…rVisitor.cpp Co-authored-by: Hugo <hugo.talbot@sofa-framework.org>
1bedcb9 to
25f28c9
Compare
|
[ci-depends-on] detected during build #10. To unlock the merge button, you must
|
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