search

KULeuvenNeuromechanics / MuscleRedundancySolver

An algorithm to estimate muscle tendon properties and/or compute muscle coordination by tracking experimental data with a musculoskeletal model assuming optimal control to solve for the muscle redundancy.
32 stars 14 forks source link

new branch #9

Closed tomvanwouwe1992 closed 4 years ago

tomvanwouwe1992 commented 4 years ago
MaartenAfschrift commented 4 years ago

Hi Tom,

Can you remove the results of the muscle analysis from this PR. Now it's hard to see the files that changed w.r.t. the master branch.

MaartenAfschrift commented 4 years ago

I think that the function (ForceEquilibrium_lMtildeState.m) is not the updated version in this PR. This should also include the new implementation with the muscle length projected on the axis of the tendon as input argument "auxk". (Maybe we should also give this variable a more meaningful name)

tomvanwouwe1992 commented 4 years ago

Ok ik zie het inderdaad. Raar, bij loopt code niet vast, ik denk dat er ergens op mijn 'path' toch al een aangepaste versie zit ofzo. En inderdaad, eigenlijk stelt dit de geprojecteerde fiber length voor?

MaartenAfschrift commented 4 years ago

@tomvanwouwe1992 adjust several things today: 1) the problem with vMtilde and lMtilde (now we implemented vMtilde as the time derivative of lMtilde). 2) cleaned up the code a little bit 3) added a minimal example in the US_tracking with only the calf muscles and tibialis anterior

Can you review these changes ? (see commit aef1dbe)

MaartenAfschrift commented 4 years ago

Tom, er is ook iets vreemds met de naamgeving in de trials van de running data. Is dit de bedoeling ? zie hier: https://github.com/tomvanwouwe1992/MuscleTendonEstimator/tree/largeUpdate/Examples/Example_TendonEstimator_RunningData/Data

Dit is niet consistent met de code: https://github.com/tomvanwouwe1992/MuscleTendonEstimator/blob/largeUpdate/Examples/Example_TendonEstimator_RunningData/Example_UStracking.m#L53

tomvanwouwe1992 commented 4 years ago

Vreemd, ik bekijk dit even en pas wel aan.

tomvanwouwe1992 commented 4 years ago

@MaartenAfschrift , ik denk dat jij de verwijzing in de code voor de running data (van Wannes) hebt aangepast naar de files van Tijs bij de commit aef1dbe.

MaartenAfschrift commented 4 years ago

Oei, ik pas dit terug aan. Het was niet de bedoeling om dit aan te passen in een vorige commit. Ik moet per ongeluk de file in de een verkeerde folder open gedaan hebben. Ik run de code nog voor de optimalisatie meeting van morgen.

MaartenAfschrift commented 4 years ago

De tracking is nu veel beter. Dit lijkt dus ok. I

TrackingNew

k voeg morgen nog een optie toe om de procentuele verandering in vezel lengte ook te koppelen voor de gastrocnemius bundels.

MaartenAfschrift commented 4 years ago

I think that we can merge the branch now. @tomvanwouwe1992 can you do a final review? I propose the create a new branch afterwards to do the final code clean-up and README file.