Skip to content

Conversation

xiangchenjhu
Copy link
Collaborator

@xiangchenjhu xiangchenjhu commented Aug 27, 2025

Description:

This PR introduces the Neo-Hookean (NH) constitutive model into the FIM inverse modeling framework.**
It extends the current material model options (previously linear and HGO) and enables parameter identification for isotropic nonlinear materials.

  1. Implemented calculate_VWS_nh following the same vectorized structure as existing models.
  2. Integrated NH into the MaterialModel interface for seamless selection and sensitivity analysis.
  3. Verified consistency with the original formulation to ensure correctness.
  4. Adds a new dataset for NH model testing (./test_data/NH).
  5. Adds an environment file (fim_env.yml) for easy reproducibility of the working environment.

This addition improves the generalizability of FIM by supporting another widely used nonlinear model, broadening applicability for soft matter characterization.

In addition, add unit tests for all three refactor modules to achieve close to 100% coverage.

Running:
image

Copy link

codecov bot commented Sep 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@xiangchenjhu xiangchenjhu changed the title Add Neo-Hookean (NH) material model to FIM Add Neo-Hookean (NH) model and standardize (x, y, z) coordinate convention in FIM Sep 16, 2025
@xiangchenjhu xiangchenjhu marked this pull request as ready for review September 18, 2025 18:03
@xiangchenjhu xiangchenjhu requested a review from a team as a code owner September 18, 2025 18:03
@xiangchenjhu xiangchenjhu marked this pull request as draft September 18, 2025 18:16
@xiangchenjhu xiangchenjhu changed the title Add Neo-Hookean (NH) model and standardize (x, y, z) coordinate convention in FIM Add Neo-Hookean (NH) model and unit tests in FIM Sep 26, 2025
@xiangchenjhu xiangchenjhu marked this pull request as ready for review September 26, 2025 19:17
@xiangchenjhu xiangchenjhu merged commit 1164809 into main Sep 26, 2025
8 checks passed
@xiangchenjhu xiangchenjhu deleted the add-NH-model branch September 26, 2025 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants