Repository navigation
Conversation
Coverage Report
The above coverage report was generated for the changes in this PR. |
…m/aryan7071/stdlib into test-migrate-bradford-entropy-ulp
|
An automated check found potentially unrelated issue/PR references in this PR:
Why this matters: GitHub automatically closes issues referenced with What to do:
This assessment was generated by an AI model and is informational only. Generated by Claude Code |
…dford-entropy-ulp
|
Thank you for working on this pull request. However, we cannot accept your contribution as the issue this pull request seeks to resolve has already been addressed in a different pull request or commit. Thank you again for your interest in stdlib, and we look forward to reviewing your future contributions. |
Resolves a part of #11352
Description
This pull request:
Measured minimum ULP bounds (identical across the main export and the native addon):
The small_c bound is large in absolute terms, consistent with the pre-existing comment in the test file: for very small values of the shape parameter c, nested ln calculations diverge slightly from the SciPy-generated reference fixtures, and the resulting entropy values are themselves very small in magnitude. Inspecting the worst-case row directly (c=0.1315...) shows the actual and expected values agree to ~13 significant digits (absolute difference ~2.15e-16); the large ULP count reflects the fine representable spacing of floating-point values at that small magnitude rather than a correctness issue.
Related Issues
This pull request has the following related issues:
math/base/specialpackages from relative tolerance testing to ULP difference testing (tracking issue) #11352Questions
No.
Other
No.
Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
{{TODO: add disclosure if applicable}}
@stdlib-js/reviewers