Skip to content

Refactor rod-plane contact examples + Continuum Snake With Lifting Wave example#317

Merged
Ali-7800 merged 11 commits into
GazzolaLab:update-0.3.2from
Ali-7800:113_dev_contact
Dec 5, 2023
Merged

Refactor rod-plane contact examples + Continuum Snake With Lifting Wave example#317
Ali-7800 merged 11 commits into
GazzolaLab:update-0.3.2from
Ali-7800:113_dev_contact

Conversation

@Ali-7800
Copy link
Copy Markdown
Collaborator

Sixth contact module PR mentioned in Issue
This PR includes:

  • Refactored examples that use AnisotropicFrictionalPlane to use RodPlaneContactWithAnisotropicFriction instead
  • Added new example: ContinuumSnakeWithLiftingWaveCase, to replicate the results from X. Zhang et. al. Nat. Comm. 2021. Related to this Issue

@Ali-7800 Ali-7800 added enhancement New feature or request prio:medium Priority level: medium labels Nov 15, 2023
@Ali-7800 Ali-7800 requested review from skim0119 and sy-cui November 15, 2023 02:09
@Ali-7800 Ali-7800 self-assigned this Nov 15, 2023
@Ali-7800 Ali-7800 removed the request for review from bhosale2 November 15, 2023 02:11
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 15, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1da4dbc) 93.46% compared to head (1273c13) 93.46%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@              Coverage Diff              @@
##           update-0.3.2     #317   +/-   ##
=============================================
  Coverage         93.46%   93.46%           
=============================================
  Files                51       51           
  Lines              3183     3183           
  Branches            348      348           
=============================================
  Hits               2975     2975           
  Misses              158      158           
  Partials             50       50           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Contributor

@armantekinalp armantekinalp left a comment

Choose a reason for hiding this comment

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

Some comments. Can you also add videos for snakes and plots for validation cases for reference.

Comment thread examples/ContinuumSnakeWithLiftingWaveCase/continuum_snake_with_lifting_wave.py Outdated
Comment thread examples/ContinuumSnakeWithLiftingWaveCase/continuum_snake_with_lifting_wave.py Outdated
Comment thread examples/ContinuumSnakeWithLiftingWaveCase/snake_forcing.py Outdated
@Ali-7800
Copy link
Copy Markdown
Collaborator Author

Ali-7800 commented Nov 28, 2023

Examples videos/plots:

MuscularSnake

2D_xy_muscular_snake.mp4

muscular_snake_velocity

Axial Friction

axial_friction

rolling friction torque

rolling_friction_torque

rolling friction on inclined plane

rolling_friction_on_inclined_plane

rolling friction initial velocity

rolling_friction_initial_velocity

Continuum snake with lifting wave

continuum_snake_with_lifting_wave.mp4

Continuum Snake

continuum_snake.mp4

![continuum_snake_velocity](https://github.com/GazzolaLab/PyElastica/assets/47090295/f403db0e-576a-4461-ac49-a6fd273e99e

continuum_snake_curvature
5)

Copy link
Copy Markdown
Contributor

@armantekinalp armantekinalp left a comment

Choose a reason for hiding this comment

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

LGTM just two comments.

Comment thread examples/ContinuumSnakeWithLiftingWaveCase/snake_forcing.py Outdated
Comment thread examples/ContinuumSnakeWithLiftingWaveCase/snake_forcing.py Outdated
Comment thread examples/ContinuumSnakeWithLiftingWaveCase/snake_forcing.py Outdated
@armantekinalp
Copy link
Copy Markdown
Contributor

@skim0119 and @sy-cui can you briefly check and approve

Comment thread examples/ContinuumSnakeWithLiftingWaveCase/snake_forcing.py Outdated
Copy link
Copy Markdown
Contributor

@sy-cui sy-cui left a comment

Choose a reason for hiding this comment

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

Minor comments

Comment thread examples/ContinuumSnakeWithLiftingWaveCase/continuum_snake_with_lifting_wave.py Outdated
Comment thread examples/ContinuumSnakeWithLiftingWaveCase/snake_contact.py
Comment thread examples/ContinuumSnakeWithLiftingWaveCase/snake_forcing.py Outdated
Comment thread examples/ContinuumSnakeWithLiftingWaveCase/snake_forcing.py Outdated
Comment thread examples/ContinuumSnakeWithLiftingWaveCase/snake_forcing.py Outdated
Comment thread examples/ContinuumSnakeWithLiftingWaveCase/snake_forcing.py Outdated
Comment thread examples/ContinuumSnakeWithLiftingWaveCase/snake_forcing.py Outdated
@Ali-7800 Ali-7800 requested a review from sy-cui December 3, 2023 23:53
Copy link
Copy Markdown
Contributor

@sy-cui sy-cui left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@skim0119 skim0119 left a comment

Choose a reason for hiding this comment

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

Looks good on my end 👍

Copy link
Copy Markdown
Contributor

@armantekinalp armantekinalp left a comment

Choose a reason for hiding this comment

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

LGTM

@armantekinalp
Copy link
Copy Markdown
Contributor

@Ali-7800 you can merge whenever you want

@Ali-7800 Ali-7800 merged commit c2a641e into GazzolaLab:update-0.3.2 Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request prio:medium Priority level: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants