-
Notifications
You must be signed in to change notification settings - Fork 317
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
[examples] Fix warnings of all examples #5183
[examples] Fix warnings of all examples #5183
Conversation
[ci-build][with-all-tests] |
Remove MakeDataAlias... it was introduce to avoid complains about removing the alias in core of Sofa. Now we agreed, we could drop it :) |
@hugtalbot yes, this will be solved (hopefully) in a few incoming PR where I totally change the incision algorithm process. |
OK, so far, the PR is therefore ready for review and then merge @damienmarchal do you confirm that both MakeAliasComponent and MakeDataAliasComponent can be removed? |
My π to SOFA users π¨π½
Obviously, this might trigger regressions π€·ββοΈ
π Some questions π
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