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

Notebooks #3847

Merged
merged 3 commits into from
Jul 10, 2020
Merged

Notebooks #3847

merged 3 commits into from
Jul 10, 2020

Conversation

vkkhare
Copy link
Member

@vkkhare vkkhare commented Jul 10, 2020

Description

speed tests
notebook auth added

Checklist

@vkkhare vkkhare requested a review from a team as a code owner July 10, 2020 12:10
@vkkhare vkkhare requested review from gmuraru and imskr and removed request for a team July 10, 2020 12:10
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

Review Jupyter notebook visual diffs & provide feedback on notebooks.


Powered by ReviewNB

@cereallarceny cereallarceny merged commit 3756d4d into OpenMined:static/notebooks Jul 10, 2020
cereallarceny added a commit that referenced this pull request Jul 10, 2020
* fix speed test and auth methods

* try to add avg plan in notebooks

* fix black issues

* Adding iterative plan setting

* Moving the notebooks to a better place

* Fix avg plan in Create Plan

* Notebooks (#3847)

* cleared speed testing

* update execute plan notebook to use auth token

* black reformat

* Adding auth

* A few changes

* Run notebooks

Co-authored-by: Patrick Cason <me@patrickcason.com>
Co-authored-by: Vova Manannikov <vova@promail.spb.ru>
Co-authored-by: varun khare <varunkhare1234@gmail.com>
Nilanshrajput pushed a commit to Nilanshrajput/PySyft that referenced this pull request Jul 15, 2020
* fix speed test and auth methods

* try to add avg plan in notebooks

* fix black issues

* Adding iterative plan setting

* Moving the notebooks to a better place

* Fix avg plan in Create Plan

* Notebooks (OpenMined#3847)

* cleared speed testing

* update execute plan notebook to use auth token

* black reformat

* Adding auth

* A few changes

* Run notebooks

Co-authored-by: Patrick Cason <me@patrickcason.com>
Co-authored-by: Vova Manannikov <vova@promail.spb.ru>
Co-authored-by: varun khare <varunkhare1234@gmail.com>
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.

2 participants