Skip to content

Restore signed scalar hypertoroidal integration - #5406

Merged
FlorianPfaff merged 2 commits into
mainfrom
chatgpt/restore-signed-torus-scalar-integration
Aug 23, 2026
Merged

Restore signed scalar hypertoroidal integration#5406
FlorianPfaff merged 2 commits into
mainfrom
chatgpt/restore-signed-torus-scalar-integration

Conversation

@FlorianPfaff

Copy link
Copy Markdown
Owner

Summary

  • restore the documented signed-integration convention for scalar 1-D hypertoroidal/circular bounds
  • keep reversed vector hyperrectangle bounds rejected
  • align the contradictory scalar-boundary regression with the established API contract

Regression fixed

The full Test workflow on the current main lineage failed across NumPy, PyTorch, and JAX. The NumPy 3.11 JUnit artifact shows two failures caused by #5401 rejecting scalar reversed bounds:

  • CircularUniformDistributionTest.test_integral_with_range
  • test_integrate_preserves_signed_scalar_boundaries

Before #5401, _validate_boundary_order explicitly exempted scalar 1-D limits because the circular API uses signed integration. This PR restores that exact compatibility rule while preserving order validation for vector hyperrectangles.

Scope

Two files only. No change to finite-value/shape validation or multidimensional boundary ordering.

@FlorianPfaff
FlorianPfaff merged commit 1f4f8f2 into main Aug 23, 2026
18 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant