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

problem_jacobi #913

Closed
wants to merge 1 commit into from
Closed

Conversation

cpz2024
Copy link

@cpz2024 cpz2024 commented Nov 12, 2024

Pull Request

在jacobi.py的函数Rotation_Matrix中,之前的写法是用J = update_J(J, t_k, t_l, cos_values[i], sin_values[i], n)来实现将J[t_k][t_l]处的值设置为cos_values[i],其中t_k和t_l是秘密共享的。实际上t_k和t_l在执行过程中是明文的,可以直接用J=J.at[t_k,t_k].set(cos_values[i]),但是这样会报错 [libspu/kernel/hal/ring.cc:63] should not be here x=Value<1x1xSF32,s=0,0>, to=Pub2k。好像是cos_values[i]的类型有问题?

Copy link


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@deadlywing deadlywing closed this Nov 18, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Nov 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants