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.

Sign up for GitHub

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

Jump to bottom

Create vertical data splitter #10

Merged
merged 4 commits into from
Jun 8, 2020
Merged

Create vertical data splitter #10

merged 4 commits into from
Jun 8, 2020

Conversation

TTitcombe
Copy link
Member

Description

This PR implements a function to vertically partition a dataset. This dataset is assumed to be an image dataset (e.g. MNIST).
Additionally, this PR introduces a conda environment, requirements.txt file

Fixes #4

How has this been tested?

  • Test environment
    1. Follow the instructions in the README to install dependencies
    2. In the environment, ensure you can import torch, torchvision, numpy, syft
  • Integrated testing of code using pytest:
    1. navigate to root of this repository
    2. Run python -m pytest. All tests should pass

@TTitcombe
Add requirements and conda env file
3de7211 
Update README with requirement installation instructions
@TTitcombe
Create basic repository structure
c805b70 
src folder for code and empty 'examples' folder for notebooks
@TTitcombe
Create function which partitions data
c95b021
@TTitcombe
Test vertical partitioning function
43008aa
@TTitcombe TTitcombe added Status: Completed ✔️ Finished with all expected deliverables Type: New Feature ➕ Introduction of a completely new addition to the codebase labels Jun 8, 2020
@pavlos-p pavlos-p merged commit cd19b1a into OpenMined:master Jun 8, 2020
@TTitcombe TTitcombe mentioned this pull request Jun 14, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@pavlos-p pavlos-p

Labels
Status: Completed ✔️ Finished with all expected deliverables Type: New Feature ➕ Introduction of a completely new addition to the codebase
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Function to vertically partition data
2 participants
@TTitcombe @pavlos-p