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

Off-by-one error in documentation for reduce32 #56

Closed
SWilson4 opened this issue Dec 22, 2021 · 2 comments
Closed

Off-by-one error in documentation for reduce32 #56

SWilson4 opened this issue Dec 22, 2021 · 2 comments

Comments

@SWilson4
Copy link

The documentation for reduce32 states that the output is in the range [-6283009, 6283007]. The range is actually [-6283009, 6283008], as evaluating reduce32 at 2^{31} - 2^{22} - 1 will illustrate.

I have a branch which fixes both this issue and #55 since they're quite minor.

@sduyyy
Copy link

sduyyy commented Jan 5, 2024

@SWilson4 how we evaluate reduce32 at 2^{31} - 2^{22} - 1 ,will you please give a detail description?

@gregorseiler
Copy link
Member

Thanks. The typos have been corrected

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 a pull request may close this issue.

3 participants