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

Express GH shoulder reactions in glenoid coordinate system #700

Merged
merged 5 commits into from
Nov 16, 2022

Conversation

melund
Copy link
Member

@melund melund commented Aug 3, 2022

The glenoid reaction forces are now epressed in the coordinate system of the glenoid cup instead of the general scapula coordinate system. This means that the three force variables GlenoHumeral_DistractionForce, GlenoHumeral_InferoSuperiorForce, GlenoHumeral_AnteroPosteriorForce now represents the three direction given by the glenoid cup.

This means that the variables will be slightly different even though the force is the same.

@menzejo Does this also make sense to you?

fixes #699

The glenoid shoulder reactions in "SelectedOutput" was previously
expressed in the scapula coordinate system.  The scapula coordinate system
was slightly misalligned with glenoid surface, which caused cross talk
between the:
GlenoHumeral_DistractionForce, GlenoHumeral_InferoSuperiorForce, andGlenoHumeral_AnteroPosteriorForce variables.

This fixes the problem by expressing the forces in the GHReactionCenterNodeRotated node.
@melund melund added the AMS7.5 label Aug 3, 2022
@melund melund changed the base branch from master to master-ammr-2.5 August 3, 2022 14:42
@melund melund requested a review from toerholm August 15, 2022 09:30
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@melund melund merged commit 0a13ee3 into master-ammr-2.5 Nov 16, 2022
@melund melund deleted the gh_shoulder_force_definition branch November 16, 2022 09:16
melund added a commit that referenced this pull request Jul 27, 2023
This is a regression from chancing the way
the force is expressed. #700

fixes: ab#2259
melund added a commit that referenced this pull request Aug 7, 2023
This is a regression from chancing the way
the force is expressed. #700

fixes: ab#2259
melund added a commit that referenced this pull request Aug 10, 2023
* Fix SternoClavicular distraction force

This is a regression from chancing the way the force is expressed. #700

fixes: ab#2259

---------

Co-authored-by: dsc <dsc@anybodytech.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Shoulder forces should be expressed in the cavity coordinate system.
1 participant