Skip to content

fix(families): raise ValueError for x outside support in builtins#118

Open
MyznikovFD wants to merge 1 commit into
mainfrom
score-function
Open

fix(families): raise ValueError for x outside support in builtins#118
MyznikovFD wants to merge 1 commit into
mainfrom
score-function

Conversation

@MyznikovFD

Copy link
Copy Markdown
Collaborator

No description provided.

@MyznikovFD MyznikovFD self-assigned this May 25, 2026
@MyznikovFD
MyznikovFD requested review from Desiment and LeonidElkin and removed request for LeonidElkin May 25, 2026 16:00
@MyznikovFD MyznikovFD added Bug Something isn't working API: Consistency Internal Consistency of API/Behavior core.families core.families.builtins labels May 25, 2026
@MyznikovFD MyznikovFD moved this to In review in PySATL Core Developing May 25, 2026
@MyznikovFD MyznikovFD linked an issue May 25, 2026 that may be closed by this pull request

@LeonidElkin LeonidElkin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The issue explicitly lists normal.py, but this commit does not add any support validation for the normal score. Even though all finite real values are in support, ContinuousSupport() excludes ±inf and NaN, and the current implementation returns inf/nan gradients for those inputs. Either add a finite-value check here or clarify why normal is intentionally out of scope.

Comment thread src/pysatl_core/families/builtins/continuous/uniform.py Outdated
Comment thread src/pysatl_core/families/builtins/continuous/exponential.py Outdated
Comment thread src/pysatl_core/families/builtins/continuous/uniform.py
Comment thread src/pysatl_core/families/parametric_family.py Outdated
@MyznikovFD
MyznikovFD force-pushed the score-function branch 3 times, most recently from b8b7845 to 61d8608 Compare July 17, 2026 13:32
@MyznikovFD
MyznikovFD requested a review from LeonidElkin July 17, 2026 14:08
@MyznikovFD MyznikovFD changed the title fix(families): raise ValueError for x outside support in exponential and uniform fix(families): raise ValueError for x outside support in builtins Jul 19, 2026
@MyznikovFD MyznikovFD mentioned this pull request Jul 19, 2026
LeonidElkin
LeonidElkin previously approved these changes Jul 19, 2026
Comment thread src/pysatl_core/families/parametric_family.py Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API: Consistency Internal Consistency of API/Behavior Bug Something isn't working core.families.builtins core.families

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

Score should raise ValueError for points outside support, not 0

2 participants