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

Merge quotient-constants into main #280

Merged
merged 1 commit into from
Feb 9, 2025
Merged

Conversation

stevehalliwell
Copy link
Owner

@stevehalliwell stevehalliwell commented Feb 9, 2025

Add PUSH_QUOTIENT.

Reworks the existing push value to store shorts not just byte(s)
Stores short / byte quotients so we can avoid having named constants for things like 0.5 or 0.3(3) or -5000.5.

close #271

@coveralls
Copy link

Coverage Status

coverage: 96.293% (+0.05%) from 96.241%
when pulling a533fe3 on quotient-constants
into 483808d on main.

@stevehalliwell stevehalliwell merged commit 1d39302 into main Feb 9, 2025
4 checks passed
@stevehalliwell stevehalliwell deleted the quotient-constants branch February 9, 2025 07:39
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.

Is it worth it to store byte quotient/floats
2 participants