-
Notifications
You must be signed in to change notification settings - Fork 275
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
Steps with neg sigma #970
base: master
Are you sure you want to change the base?
Steps with neg sigma #970
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #970 +/- ##
==========================================
+ Coverage 93.13% 93.24% +0.10%
==========================================
Files 10 10
Lines 3787 3789 +2
==========================================
+ Hits 3527 3533 +6
+ Misses 260 256 -4 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks @newville a few minor typos perhaps but other than that looks good to me. Perhaps just squash the commits?
Starts at 0.0, ends at `amplitude`, with half-max at `center`, and | ||
rising with `form`: | ||
Starts at 0.0, ends at `sign(sigma)*amplitude`, has a half-max at | ||
`center`, rsing or falling with `form`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo: rsing --> rising
:math:`A` rising to :math:`A/2` at :math:`\mu`, with :math:`\sigma` | ||
setting the characteristic width. The functional forms are defined as: | ||
:math:`\tt{sign}(\sigma)A` rising or falling to :math:`A/2` at :math:`\mu`, | ||
with :math:`\sigma` setting the characteristic width and the sign up the step. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"sign up" is that what you meant?
Description
This allows the
step
lineshape andStepModel
to have values ofsigma<0
to signify a step down (from 1 to 0).See discussion at: https://groups.google.com/g/lmfit-py/c/aBBzQG0uDK4
Type of Changes
Tested on
Python: 3.12.5 | packaged by conda-forge | (main, Aug 8 2024, 18:31:54) [Clang 16.0.6 ]
lmfit: 1.3.2.post9+g776e14d7.d20241007, scipy: 1.14.0, numpy: 1.26.4, asteval: 1.0.5, uncertainties: 3.2.2.post7+gce16269
Verification
Have you