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

Remove hard-coded SplitNN #80

Open
TTitcombe opened this issue Dec 9, 2020 · 0 comments
Open

Remove hard-coded SplitNN #80

TTitcombe opened this issue Dec 9, 2020 · 0 comments
Labels
Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Type: Improvement 📈 Performance improvement not introducing a new feature or requiring a major refactor
Milestone
Pip package

Comments

@TTitcombe
Copy link
Member

Description

There is a hardcoded SplitNN in the source code. This should be removed.

Are you interested in working on this improvement yourself?

  • Yes, I am, but anyone else is free to take this

Additional Context

We should remove this before we think about publishing the first PyVert pip package
@TTitcombe TTitcombe added Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Type: Improvement 📈 Performance improvement not introducing a new feature or requiring a major refactor labels Dec 9, 2020
@TTitcombe TTitcombe added this to the Pip package milestone Dec 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Type: Improvement 📈 Performance improvement not introducing a new feature or requiring a major refactor
Projects
None yet
Milestone
Pip package
Development

No branches or pull requests
1 participant
@TTitcombe