-
Notifications
You must be signed in to change notification settings - Fork 4
Walking reconstruction example #54
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 106451 lines exceeds the maximum allowed for the inline comments feature.
@Ipuch : My bad, we fixed my environment problem together. So this was already OK and reviewed. |
@EveCharbie can you try to merge all vtp in one folder ? To avoid duplication of .vtp files ? make sure my msk example still work. |
Could you please explain a bit more ? |
I mean I have a folder for meshes for my upper limb example. Some of the vtp files are the same. To avoid the redundancy we could merge the folders and reduce the size of the Pr/repo git history. I also means your pr needs a force-push. |
I don't understand since I already put my vtp files in your folder examples/biorbd/models/Geometry_cleaned. |
okay for the redundancy. The first part of your last answer, made it clearer that files are not the same for your example and mine.
Do you have these problems locally too ? |
Code Climate has analyzed commit 44ca9cd and detected 0 issues on this pull request. View more on Code Climate. |
Unfortunately, those are problems related to osim_to_biomod :( |
This change is