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

Add Scikit-Learn to Notebook requirements.txt #3339

Merged
merged 1 commit into from
Apr 12, 2020

Conversation

ucalyptus
Copy link
Contributor

@ucalyptus ucalyptus commented Apr 11, 2020

As per #3272 , there is a dependency missing issue while running Tutorial 5 notebook.

@ucalyptus ucalyptus requested a review from a team as a code owner April 11, 2020 18:38
@ucalyptus ucalyptus changed the title Based on #3272 Add Scikit-Learn to Notebook requirements.txt Apr 11, 2020
@codecov
Copy link

codecov bot commented Apr 11, 2020

Codecov Report

Merging #3339 into master will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3339      +/-   ##
==========================================
+ Coverage   94.65%   94.68%   +0.04%     
==========================================
  Files         151      151              
  Lines       16240    16240              
==========================================
+ Hits        15370    15375       +5     
+ Misses        870      865       -5     
Impacted Files Coverage Δ
syft/generic/pointers/object_pointer.py 81.96% <0.00%> (+3.76%) ⬆️

@ucalyptus
Copy link
Contributor Author

@karlhigley

@karlhigley karlhigley merged commit ce37c64 into master Apr 12, 2020
@karlhigley karlhigley deleted the ucalyptus-scikit-add branch April 12, 2020 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants