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

Optimize garbling control flow #142

Open
sinui0 opened this issue May 28, 2024 · 6 comments
Open

Optimize garbling control flow #142

sinui0 opened this issue May 28, 2024 · 6 comments
Labels
good first issue Good for newcomers

Comments

@sinui0
Copy link
Collaborator

sinui0 commented May 28, 2024

A significant portion of the garbling/evaluation time in our implementation is spent on branching while iterating over the gates of a circuit. We should optimize it without using unsafe.

@sinui0 sinui0 added the good first issue Good for newcomers label May 28, 2024
@m-rit
Copy link

m-rit commented Jun 6, 2024

Hi @sinui0, can I raise a PR to fix this issue ? I am currently exploring this library and would love to contribute to the issues!

@sinui0
Copy link
Collaborator Author

sinui0 commented Jun 6, 2024

Certainly 👍

Please base on the threading-refactor branch for now, we have numerous changes coming.

@yawn-c111
Copy link

@sinui0

Hi!
I'm a participant in the PSE Core Program 2024.
I'm learning about MPC and garbled circuits, and although this issue seems quite challenging for my current level, I'm interested in taking it on as a learning opportunity.

Would it be alright if I try to tackle this issue?

Since the threading-refactor branch has already been merged into dev, I was thinking of creating a new branch from dev to work on this.
Would that be the correct approach?

@themighty1
Copy link
Collaborator

hi @yawn-c111 , certainly, this issue is still up for grabs. We haven't heard from @m-rit , so it is safe to assume they are not working on this anymore. Feel free to start working on this. We are also available on DIscord during work hours. Thanks.

@yawn-c111
Copy link

@themighty1
Thank you for your reply!
I would like to start working on it!

@yawn-c111
Copy link

@themighty1
I took you up on your kind offer and asked a question about this issue in the mpz thread on PSE's Discord!
I would appreciate it if you could check it when you have time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants