-
Notifications
You must be signed in to change notification settings - Fork 5
examples: Add diffpy.srfit examples
#92
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
Conversation
|
@sbillinge ready for review. fyi, srfit had a |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #92 +/- ##
==========================================
+ Coverage 92.78% 93.63% +0.85%
==========================================
Files 3 4 +1
Lines 194 220 +26
==========================================
+ Hits 180 206 +26
Misses 14 14 🚀 New features to boost your workflow:
|
|
Added one thing to docs. ready for review |
|
@sbillinge please see the other PR here #94 for more discussion on this test. |
closes #4
tested each example by copying to cwd with
cmi copy coreandcmi copy nanoparticle-fitand everything ran as intended.current cmi examples are:
This also adds the new module
diffpy.cmi.fit_toolswhere we house a lightweight optimization wrapper and a plotting function. in the future, we can add different optimizers to the optimization function, but for now it just hasscipy.leastsq.