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

Apply PSI to SplitNN training example #34

Closed
TTitcombe opened this issue Jul 1, 2020 · 0 comments · Fixed by #39
Closed

Apply PSI to SplitNN training example #34

TTitcombe opened this issue Jul 1, 2020 · 0 comments · Fixed by #39
Assignees
Labels
Type: New Feature ➕ Introduction of a completely new addition to the codebase

Comments

@TTitcombe
Copy link
Member

TTitcombe commented Jul 1, 2020

Feature Description

Apply the PSI code to the SplitNN notebook to re-link vertically partitioned data. In its current state, the partitioned data is ordered, so PSI is unnecessary; change the arguments passed to the partition function to shuffle the partitioned data.

  • Change the parameters passed to the partition function so partitioned data is unordered
  • Demonstrate in the notebook that data is unordered i.e. print part of the data (this is not functional, but it helps to tell the story of what we're trying to achieve)
  • Use the PSI code to re-order the data
  • Train the SplitNN

Additional Context

Incorporating PSI into the SplitNN training example will finish the PoC
Do not work on this issue until #32 refactor is complete

@TTitcombe TTitcombe added the Type: New Feature ➕ Introduction of a completely new addition to the codebase label Jul 1, 2020
@TTitcombe TTitcombe mentioned this issue Jul 1, 2020
Closed
5 tasks
@pavlos-p pavlos-p self-assigned this Jul 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: New Feature ➕ Introduction of a completely new addition to the codebase
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants