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

Moved schemeswitching-data-serializer.h from pke/include/scheme/ckksrns to pke/unittest as it is a collection of special utilities #916

Merged
merged 2 commits into from
Feb 5, 2025

Conversation

dsuponitskiy
Copy link
Collaborator

No description provided.

…ns to pke/unittest as it is a collection of special utilities
@dsuponitskiy dsuponitskiy force-pushed the 869-move-schemeswitching-serializer branch from d23c870 to 2333801 Compare January 24, 2025 16:57
Copy link
Collaborator

@andreea-alexandru andreea-alexandru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but please add a paragraph in SCHEME_SWITCHING_CAPABILITY.md to say that an example for serializing scheme switching is given in scheme-switching-serial.cpp and users should also update src/pke/unittest/utils/schemeswitching-data-serializer.h if they want to specify other filenames.

Copy link
Collaborator

@andreea-alexandru andreea-alexandru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to go.

@pascoec pascoec merged commit 0ee54e7 into tmp-dev Feb 5, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Code cleanup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move schemeswitching-data-serializer.h from pke/include to pke/unittest/ or pke/examples.
4 participants