Skip to content

Commit

Permalink
fix: Cervical spine axial rotation rhythm coefficient fix (#23)
Browse files Browse the repository at this point in the history
* Bug fix cervical spine axial rotation rhythm coefficient. The distribution between C2C1 and rest of cervical spine was incorrect.

* update changelog

* Apply suggestion to use coefficients from rhythm data file.

Co-authored-by: Morten Enemark Lund <[email protected]>
Signed-off-by: divyaksh-chander <[email protected]>

---------

Signed-off-by: divyaksh-chander <[email protected]>
Co-authored-by: Morten Enemark Lund <[email protected]>
  • Loading branch information
divyaksh-chander and melund authored Sep 28, 2024
1 parent 7cd0b61 commit 1b487a9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
3 changes: 1 addition & 2 deletions Body/AAUHuman/Trunk/JointsCervical.any
Original file line number Diff line number Diff line change
Expand Up @@ -219,8 +219,7 @@ AnyFolder Rotation = {

RhythmDriverLinear rhythmC2C1SkullThoraxRotation( _REDEFINE_VARIABLES=On) =
{
// The -1 is because C2C1 is a revolute jnt and has the opposite signs for output.
AnyVector RhythmCoefficients ??= {132/77, -1};
AnyVector RhythmCoefficients ??= {0.2721+0.3061, 0.2721};

Measures.Input = {
AnyKinMeasure& ref = ...SkullThoraxRotation;
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ The default pelvis model used in all models have changed. The pelvis morphology
**Fixed:**

* Fixed the inclusion of the buckle segmental masses in the calculation of the TotalBodyMass variable.
* Fixed the cervical spine axial rotation rhythm coefficients. The distribution between C2C1 and the rest of cervical spine was incorrect.

**Added:**

Expand Down

0 comments on commit 1b487a9

Please sign in to comment.