Skip to content

fix: Muscle renaming#146

Open
JacobAdamsen wants to merge 7 commits intoammr4-betafrom
MuscleNaming
Open

fix: Muscle renaming#146
JacobAdamsen wants to merge 7 commits intoammr4-betafrom
MuscleNaming

Conversation

@JacobAdamsen
Copy link
Contributor

Further work of the muscle renaming done in PR #97 and #96.
Renaming muscles to follow the same templates/conventions as in the previous 2 PR's.
General review of muscles that still needs to be renamed.

@JacobAdamsen JacobAdamsen requested a review from bkje14 February 28, 2026 14:13
@JacobAdamsen JacobAdamsen changed the title Muscle renaming fix: Muscle renaming Feb 28, 2026
@github-actions github-actions bot added the fix label Feb 28, 2026

This comment was marked as resolved.

Copy link
Contributor

@bkje14 bkje14 left a comment

Choose a reason for hiding this comment

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

I can help you make the PR for updating the muscle parameter files - that I think we should do before merging this PR.


#if BM_ARM_DELTOID_WRAPPING == OFF
AnyFolder deltoideus_scapular_part_1 =
AnyFolder DeltoideusScapular1 =
Copy link
Contributor

Choose a reason for hiding this comment

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

This whole file is generated from a template so we need to update the data to the template - I can show you where it is :)

#if BM_ARM_DELTOID_WRAPPING == OFF
/////////////////////////////////////////////////////
// deltoideus_scapular_part_2 (1 element ) //
// DeltoideusScapular2 (1 element ) //
Copy link
Contributor

Choose a reason for hiding this comment

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

This whole file is generated from a template so we need to update the data to the template - I can show you where it is :)
The templates and data is in the same repo as you did the BikeModel updates

AnyFolder QuadratusLumborumR12IC_1 =
{
AnyString Muscle = "QLC12Lat_CILat"; ///< Descriptive name for the muscle, e.g. for plotting
AnyString Muscle = "QuadratusLumborumR12Lat_CILat"; ///< Descriptive name for the muscle, e.g. for plotting
Copy link
Contributor

Choose a reason for hiding this comment

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

@JacobAdamsen can you help determine if "Lat" is most often written out as "Lateral" in the code ?

AnyFolder QuadratusLumborumR12IC_2 =
{
AnyString Muscle = "QLC12InterMedian_CILat"; ///< Descriptive name for the muscle, e.g. for plotting
AnyString Muscle = "QuadratusLumborumR12InterMedian_CILat"; ///< Descriptive name for the muscle, e.g. for plotting
Copy link
Contributor

Choose a reason for hiding this comment

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

This file is also auto generated from data

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.

3 participants