dilithium icon indicating copy to clipboard operation
dilithium copied to clipboard

Off-by-one error in documentation for reduce32

Open SWilson4 opened this issue 2 years ago • 1 comments

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.

SWilson4 avatar Dec 22 '21 17:12 SWilson4

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

sduyyy avatar Jan 05 '24 05:01 sduyyy