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

Support Arrays Up to U1277 #78

Closed
oddcoder opened this issue Jun 27, 2024 · 1 comment
Closed

Support Arrays Up to U1277 #78

oddcoder opened this issue Jun 27, 2024 · 1 comment

Comments

@oddcoder
Copy link

oddcoder commented Jun 27, 2024

This is a bit annoying I can imagine but NTRU-prime performs ring operation on polynomials of lengths that can go up to 1277

so naturally, it will be nice if hybrid arrays can support that. There is another problem which is which is U1277 is not even part of typenum so we can open PR there to extend the numbers in there, or we can add the numbers locally to hybrid-array so we can implement the appropriate trait.

If you can accept numbers up to U1277 to be defined in hybrid array along side the respective derives. I can roll up a quick PR tonight actually.

To observe the usage of such number check possible values of p here then you can check the functions that operate on arrays of size p such as R3_mult here.

If I would add the arrays here, I will add them under custom use flags so they don't have to slow the build time of everyone else.

@tarcieri
Copy link
Member

There is another problem which is which is U1277 is not even part of typenum

That's not a problem. We already support sizes not defined in typenum.

It would be good to discuss this under #66, the existing issue for tracking adding additional array sizes.

Closing as dup.

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

No branches or pull requests

2 participants