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

108 contact examples and fixes #109

Conversation

armantekinalp
Copy link
Contributor

Fixes #108 and also we added example cases for rod-cylinder contact.

@armantekinalp armantekinalp added bug Something isn't working enhancement New feature or request labels Jun 11, 2022
@armantekinalp armantekinalp self-assigned this Jun 11, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jun 11, 2022

Codecov Report

❗ No coverage uploaded for pull request base (update-0.2.4@a511e49). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             update-0.2.4     #109   +/-   ##
===============================================
  Coverage                ?   87.09%           
===============================================
  Files                   ?       40           
  Lines                   ?     2534           
  Branches                ?      343           
===============================================
  Hits                    ?     2207           
  Misses                  ?      305           
  Partials                ?       22           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a511e49...2997d10. Read the comment docs.

@armantekinalp
Copy link
Contributor Author

Flags capitalized and post-processing scripts are moved to post_processing.py file.

Copy link
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.

LGTM

@armantekinalp armantekinalp merged commit d76072e into GazzolaLab:update-0.2.4 Jun 12, 2022
@armantekinalp armantekinalp deleted the 108_contact_examples_and_fixes branch June 12, 2022 01:39
pppprub pushed a commit to pppprub/PyElastica that referenced this pull request Sep 4, 2022
**Update**
* We decided to have `numba` as our core dependency, and drop numpy-only version.
* Update merge existing `numpy` functions into `elastica` module. If `numba` implementation exist, `numba` version is merged and existing `numpy` version is discarded.
* Left `_elastica_numba` module and references, but throws deprecation warning if one tries to load.
* `_elastica_numpy` module throws deprecation warning.
* Impacted test cases are modified.

**Module**
* boundary condition
* external_forces
* external_forces
* interaction
* joint
* reset_ghost_vector_or_scalar
* rotation
* rod
* timestepper

**Modified**
* remove Numpy code for Muscular flagella
This commit removes Numpy code for connections and forcing for muscular
flagella, and renames the Numba code as `connection_flagella.py` and
`muscle_forces_flagella.py` .
* typo in muscular flagella connections fixed
This commit fixes the typo in `MuscularFlagellaConnection` class.
* fix numpy-int depreciate warning
This commit changes np.int to np.int64 and np.complex to np.complex128
in `memory_block_rod.py` and `analytical.py` . Since Numpy will
depreciate them in future releases.

Co-authored-by: Seung Hyun Kim <skim449@illinois.edu>
Co-authored-by: Bhosale <bhosale2@illinois.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants