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.

Sign up for GitHub

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

Jump to bottom

CKKS encoder checks for IsNTT #538

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

CKKS encoder checks for IsNTT #538

wants to merge 1 commit into from

Conversation

lehugueni
Copy link
Contributor

Fixes #501. The CKKS encoder checks the value of pt.IsNTT applies the NTT only if true.

@lehugueni lehugueni added bug Something isn't working quick-review labels Jan 23, 2025
@lehugueni lehugueni self-assigned this Jan 23, 2025
@lehugueni lehugueni linked an issue Jan 23, 2025 that may be closed by this pull request
[Enhancement] The encoder in CKKS is required for the IsNTT checking. #501
Open
@lehugueni lehugueni requested a review from MGeorgie January 23, 2025 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Pro7ech Pro7ech Pro7ech approved these changes

@MGeorgie MGeorgie Awaiting requested review from MGeorgie

At least 1 approving review is required to merge this pull request.

Assignees

@lehugueni lehugueni

Labels
bug Something isn't working quick-review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Enhancement] The encoder in CKKS is required for the IsNTT checking.
2 participants
@lehugueni @Pro7ech