Skip to content

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

Merged
pascoec merged 2 commits intotmp-devfrom
869-move-schemeswitching-serializer
Feb 5, 2025
Merged

Moved schemeswitching-data-serializer.h from pke/include/scheme/ckksrns to pke/unittest as it is a collection of special utilities#916
pascoec merged 2 commits intotmp-devfrom
869-move-schemeswitching-serializer

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
@dsuponitskiy dsuponitskiy deleted the 869-move-schemeswitching-serializer branch March 26, 2025 17:11
yspolyakov pushed a commit that referenced this pull request Apr 2, 2025
…ns to pke/unittest as it is a collection of special utilities (#916)

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

* Addressed code review comments

---------

Co-authored-by: Dmitriy Suponitskiy <dsuponitskiy@dualitytech.com>
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