⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Fix edge case in analytical integral of exponential TF1 #20992

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.

Sign up for GitHub

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

Jump to bottom
Open
swetank18 wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix edge case in analytical integral of exponential TF1 #20992

swetank18 wants to merge 1 commit into from

Conversation

@swetank18
Copy link

@swetank18 swetank18 commented Jan 22, 2026

Motivation:
The analytical integral for exponential TF1 functions does not handle the
p1 -> 0 limit and may result in division by zero or numerical instability.

Changes:

  • Handle the p1 == 0 case using the correct analytical limit
  • Preserve existing behavior for non-zero slopes

Impact:

  • Improves numerical robustness of TF1 analytical integrals
  • No API changes

@swetank18 swetank18 requested a review from hageboeck as a code owner January 22, 2026 18:57
@swetank18 swetank18 mentioned this pull request Jan 22, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hageboeck hageboeck Awaiting requested review from hageboeck hageboeck is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@hageboeck hageboeck

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@swetank18 @hageboeck