Skip to content
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

Achilles tendon wrapping 2 #788

Merged
merged 3 commits into from
Apr 21, 2023
Merged

Conversation

melund
Copy link
Member

@melund melund commented Apr 20, 2023

@depierie I made a new PR with a cleaned-up version of #766

I have removed the extra calibration and custom tibial rotation. That should probably have its own PR.

I have made the improvements exclusive to TLEM-2.0. I think that is best. It will limit the amount of validation we will have to do.

@melund melund changed the base branch from master to master-ammr-2.5 April 20, 2023 09:14
@melund
Copy link
Member Author

melund commented Apr 20, 2023

@depierie Here is branch on top of this one, which have your calibration work and the custom tibial rotation:
https://github.com/AnyBody/ammr/tree/enrico-calibration-and-tibial-rotation

Co-authored-by: Enrico De Pieri <[email protected]>
@melund melund force-pushed the achilles-tendon-wrapping-2 branch from bc6fdea to ab5671d Compare April 20, 2023 10:25
CHANGELOG.md Outdated
Comment on lines 52 to 58
* Wrapping surfaces have been added to the achilles tendon around the ankle in
the TLEM 2.0 leg model. This ensures a even ratio of moment arms between
the soleus and gastrocnemius muscles. Hence, gastroc is recruited less in
especially downhill walking, solving the models tendency to overpredict the
knee contact forces at toe off. This is the first of a number of improvements
to the leg model by Dr. Enrico De Pieri who is working on a
publication on improvements and validation of the TLEM 2 leg model.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@depierie I have added a entry to the changelog. Is this OK, otherwise just let me know what to write instead

@melund melund merged commit b93952b into master-ammr-2.5 Apr 21, 2023
@melund melund deleted the achilles-tendon-wrapping-2 branch April 21, 2023 12:00
melund added a commit that referenced this pull request Apr 27, 2023
This reverts commit b93952b.

Revert "Add intial wrapping guess for calf muscles"

This reverts commit 771fe9d.
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.

2 participants