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

feat: adapt to interconnection transport messages #377

Closed
wants to merge 1 commit into from

Conversation

Surfacebuaa
Copy link

Changes:

  1. adapt to interconnection transport messages

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


Surfacebuaa seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@Jamie-Cui
Copy link
Member

Hi, you need to sign our Contributor License Agreement before we can accept your contribution

@Candicepan
Copy link
Contributor

Thank you for your attention to SecretFlow. I would like to further communicate with you to see more cooperation possibilities. If it is convenient, you can add WeChat:SecretFlow04

@Surfacebuaa
Copy link
Author

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

@huocun-ant
Copy link
Contributor

你好,请问能提供一下这个PR的背景吗?比如:

  1. 该PR适配于哪个调度框架,哪个版本呢?或者说这个修改是在哪个框架上面调通的。
  2. 我们之前的协议是和洞见调通的,想问一下这个修改有标准文档可以参考吗

@Surfacebuaa
Copy link
Author

Surfacebuaa commented Sep 3, 2024

你好,请问能提供一下这个PR的背景吗?比如:

  1. 该PR适配于哪个调度框架,哪个版本呢?或者说这个修改是在哪个框架上面调通的。
  2. 我们之前的协议是和洞见调通的,想问一下这个修改有标准文档可以参考吗

  1. 适配过蓝象的调度层 https://github.com/be-io/mesh ,与FATE 2.0的调度层OSX protobuf也一致,参考 https://github.com/FederatedAI/FATE/blob/master/proto/osx/osx.proto
  2. protobuf根据金科联盟互联互通标准传输层push、pop接口修改:https://github.com/secretflow/InterOp/blob/main/%E4%BA%92%E8%81%94%E4%BA%92%E9%80%9AAPI%E6%8E%A5%E5%8F%A3%E8%A7%84%E8%8C%83/4.%E4%BC%A0%E8%BE%93%E5%B1%82%E6%8E%A5%E5%8F%A3/%E9%9A%90%E7%A7%81%E8%AE%A1%E7%AE%97%E4%BA%92%E8%81%94%E4%BA%92%E9%80%9A%E4%BC%A0%E8%BE%93%E5%B1%82API.md#321-%E6%95%B0%E6%8D%AE%E5%8F%91%E9%80%81%E6%8E%A5%E5%8F%A3
  3. 这么修改是否影响洞见调度层?应该是不影响互联互通协议参考实现这个仓库 https://github.com/secretflow/interconnection-impl

Copy link

github-actions bot commented Oct 3, 2024

Stale pull request message. Please comment to remove stale tag. Otherwise this pr will be closed soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants