Inversed-Tech / eyelid

Private iris matching
Apache License 2.0
0 stars 0 forks source link

Do more tests on polynomial multiplication #12

Closed teor2345 closed 3 months ago

teor2345 commented 5 months ago

In #9 we're going to modify the polynomial multiplication algorithm.

But we don't have comprehensive tests yet, we're missing some edge cases like:

And we don't have full coverage of our implementation:

We might also want to:

Let's implement a reasonable number of edge cases, then close this ticket. We don't need to do them all.

teor2345 commented 3 months ago

We’re pretty confident this works, so I don’t think we need extra tests.