Skip to content
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.

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

feat(397): Add min and max value helpers for builtin types #398

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

joshua-spacetime
Copy link
Collaborator

Fixes #397.

Adding these helpers for computing multidimensional index bounds. Any unbounded or infinite dimensions need to be made finite by computing the min or max value for the type of said dimension.

For example

a < 5 and b < 6

needs to become

a >= MIN and a < 5 and b >= MIN and b < 5

Description of Changes

API and ABI

  • This is a breaking change to the module ABI
  • This is a breaking change to the module API
  • This is a breaking change to the ClientAPI
  • This is a breaking change to the SDK API

If the API is breaking, please state below what will break

crates/sats/src/builtin_type.rs Outdated Show resolved Hide resolved
crates/sats/src/builtin_type.rs Outdated Show resolved Hide resolved
crates/sats/src/builtin_type.rs Outdated Show resolved Hide resolved
@joshua-spacetime joshua-spacetime force-pushed the joshua/397/max-values-for-builtin-types branch from f14b198 to bc818d6 Compare October 10, 2023 14:50
@joshua-spacetime joshua-spacetime force-pushed the joshua/397/max-values-for-builtin-types branch from bc818d6 to 225320e Compare October 10, 2023 15:35
Fixes #397.

Adding these helpers for computing multidimensional index bounds.
Any unbounded or infinite dimensions need to be made finite by computing
the min or max value for the type of said dimension.

For example
```
a < 5 and b < 6
```
needs to become
```
a >= MIN and a < 5 and b >= MIN and b < 5
```
@joshua-spacetime joshua-spacetime force-pushed the joshua/397/max-values-for-builtin-types branch from 225320e to 714ac21 Compare October 10, 2023 15:38
@joshua-spacetime joshua-spacetime enabled auto-merge (squash) October 10, 2023 15:59
@joshua-spacetime joshua-spacetime merged commit c1fd28a into master Oct 10, 2023
5 checks passed
@joshua-spacetime joshua-spacetime deleted the joshua/397/max-values-for-builtin-types branch October 12, 2023 22:07
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.

Add helpers for computing the min and max values of numeric builtin types
2 participants