Skip to content

Fix SciPy integration and guard bulk modulus fits - #7

Merged
gmp007 merged 3 commits into
mainfrom
fix-scipy-fit-guards
Jun 11, 2026
Merged

gmp007 merged 3 commits into
mainfrom
fix-scipy-fit-guards

Conversation

@gmp007

@gmp007 gmp007 commented Jun 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • replace the removed scipy.integrate.trapz import with trapezoid
  • validate Birch-Murnaghan fit inputs before calling curve_fit so too-few data points produce a clear message instead of an uncaught SciPy error
  • add lightweight unittest and CI checks that do not install the full scientific/ML dependency stack

Testing

  • python3 -m unittest discover -s tests
  • python3 -m py_compile src/process_stress.py

Addresses #5.

@gmp007
gmp007 marked this pull request as ready for review June 11, 2026 16:06
@gmp007
gmp007 merged commit 4868996 into main Jun 11, 2026
1 check passed
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.

1 participant