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

CKKSVector: use multithreading for sum_inplace #243

Open
bcebere opened this issue Mar 21, 2021 · 2 comments
Open

CKKSVector: use multithreading for sum_inplace #243

bcebere opened this issue Mar 21, 2021 · 2 comments
Labels
Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Type: New Feature ➕ Introduction of a completely new addition to the codebase

Comments

@bcebere
Copy link
Member

bcebere commented Mar 21, 2021

Description

The sum_inplace operation of the CKKSVector can be done using multithreading.

More precisely, we can parallelize this loop.

You can see an example of how to use the existing threadpool here.
@bcebere bcebere added Type: Improvement 📈 Performance improvement not introducing a new feature or requiring a major refactor Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Type: New Feature ➕ Introduction of a completely new addition to the codebase and removed Type: Improvement 📈 Performance improvement not introducing a new feature or requiring a major refactor labels Mar 21, 2021
@CerineBnsd
Copy link
Contributor

CerineBnsd commented May 6, 2021 •
edited
Loading

The code will be the same in both CKKSVector and BFVVector (#242), is it possible to write it only once in encrypted_vector.h ?

@youben11
Copy link
Member

youben11 commented May 8, 2021

Yes, it totally make sense if it's the same on both sides

@vdasu vdasu mentioned this issue Apr 27, 2022
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Type: New Feature ➕ Introduction of a completely new addition to the codebase
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bcebere @youben11 @CerineBnsd