Skip to content

fixes to from_array issue #335 - #338

Open
warshqm wants to merge 2 commits into
mainfrom
from_array_fix
Open

fixes to from_array issue #335#338
warshqm wants to merge 2 commits into
mainfrom
from_array_fix

Conversation

@warshqm

@warshqm warshqm commented Mar 30, 2026

Copy link
Copy Markdown

Fixes related to issue 335.

  • Fixed value logspace is simulated and checked beyond the executed values to prevent overflows from tricking the conditionals.
  • Hard check of step sizes.
  • Enacted similar warnings for linear spaces.
  • Integer logspace warning added to prevent issues on fixed overflow for Cast.mul_int_by_fixed().
  • Fixed the range check for all fixed: was previous an and when it should have been or to raise the warning.

Validated fixed and int linear and log from_arrays. Test file is available, but not added to repo.

additional warnings for log overflow, logspace fixed overflow, linear overflow, and logspace final iteration overflow
@warshqm
warshqm requested review from HiroQM and KevinAVR March 30, 2026 17:07
@github-actions

github-actions Bot commented Mar 30, 2026

Copy link
Copy Markdown

Unit Test Results

0 tests   0 ✔️  0s ⏱️
0 suites  0 💤
0 files    0

Results for commit b9e4da8.

♻️ This comment has been updated with latest results.

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