Conversation
Ipuch
left a comment
There was a problem hiding this comment.
@Ipuch reviewed 18 of 18 files at r1, all commit messages.
Reviewable status: all files reviewed, 5 unresolved discussions (waiting on @p-shg)
bioptim/examples/holonomic_constraints/arm26_pendulum_swingup_Muscle_algebraic.py line 1 at r1 (raw file):
"""
"muscle_algebraic.py"
bioptim/examples/holonomic_constraints/four_bar.py line 1 at r1 (raw file):
"""
add the test somewhere.
bioptim/examples/holonomic_constraints/custom_dynamics.py line 166 at r1 (raw file):
class HolonomicMusclesDynamics(HolonomicTorqueDynamics):
put everything in custom_dynamics_muscle.py, copy paste or import from the other file.
bioptim/examples/holonomic_constraints/arm26_pendulum_swingup.py line 1 at r1 (raw file):
"""
Not necessary
bioptim/examples/holonomic_constraints/arm26_pendulum_swingup_Muscle.py line 1 at r1 (raw file):
"""
name of the file "_muscle.py"
Specifically using more objects, or more than one constraint, on different objects or on the same object pair. This should help newer users of holonomic constraints understand this better
Specifically using more objects, or more than one constraint, on different objects or on the same object pair. This should help newer users of holonomic constraints understand this better
TODO add tests
Ipuch
left a comment
There was a problem hiding this comment.
@Ipuch partially reviewed 26 files and all commit messages, and resolved 5 discussions.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @p-shg).
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #1024 +/- ##
==========================================
- Coverage 78.62% 77.15% -1.47%
==========================================
Files 159 193 +34
Lines 18608 21019 +2411
==========================================
+ Hits 14631 16218 +1587
- Misses 3977 4801 +824
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
All Submissions:
New Feature Submissions:
black . -l120 --exclude "external/*")?Changes to Core Features:
Working on Muscle driven dynamics for models with holonomic constraints
@pariterre the arm26_pendulum_swingup_Muscle_algebraic.py file reproduces the plotting error I told you about
This change is