-
Notifications
You must be signed in to change notification settings - Fork 102
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
[sml] add PCA in jax #240
[sml] add PCA in jax #240
Conversation
sorry,我跑了一下你的单元测试,差别有点过于大了。我看了一下您的实现,似乎fit有点问题,需要修改一下哈。 |
另外,抱歉之前写要求的时候遗漏了, Thanks |
您好,可以给出具体的精度标准吗?最好能有实例对比,这样我可以在本地测试好在提交,谢谢! |
Thanks |
ok, 我用迭代法修改了一下 PCA 类,现在做到精度在1e-4以内了,请您看一下。 |
在高维数据上测了一下,应该可以达到1e-3的精度,麻烦您确认一下。 |
算法实现应该是ok的,个人感觉应该没啥大问题了,我稍后在开发机上跑一下emulation。 提几个小建议:
Thanks |
您好,感谢您的建议,我已经在代码中做出相应修改,麻烦您再看一下,谢谢! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
您好,我最近发现之前emul部分的代码有点问题,即在运行emul程序之前,是需要先将数据load到PYU上的,但之前emulation并没有提供相应的api;所以如果使用了自定义数据,实际上emul部分都是在明文下运行的,辛苦您再发起一次PR,修复一下这个问题,具体可以参考:(其实主要就是调用一下 https://github.com/secretflow/spu/blob/main/sml/linear_model/simple_sgd_emul.py |
[Solve] 使用 SPU 实现主成分分析基础功能