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

added introduction blog documentation #274

Merged
merged 2 commits into from
Aug 29, 2020

Conversation

shivaylamba
Copy link
Contributor

Description

Converted the article (https://docs.google.com/document/d/17DseJokINbxJbBTBMJursrnUcvv_pgOeDQDtVTDHGpU/edit) written by @TravellerOnCode into rst format to be added to the docs.
Please review @chinmayshah99

Affected Dependencies

NA

How has this been tested?

Tested by running on online rst viewers and editors.

Checklist

chinmayshah99
chinmayshah99 previously approved these changes Aug 27, 2020
@chinmayshah99 chinmayshah99 added the Status: Review Needed 🙋 This needs someone to approve, deny, comment, or request changes label Aug 27, 2020
midopooler
midopooler previously approved these changes Aug 27, 2020
Copy link
Member

@midopooler midopooler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

:alt: netflix
:align: center

< a click from Secure AI Course>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image Credits: Secure and Private AI (Udacity)

:alt: local
:align: center

(from google images)
Copy link

@TravellerOnCode TravellerOnCode Aug 27, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image Credit: Google Images


SOME OTHER LIBRARIES FOR DP

* `OpenDp <https://github.com/opendifferentialprivacy>`_ by Harvard University

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OpenDp by Harvard University and Microsoft


.. figure:: https://user-images.githubusercontent.com/19529592/91377299-b58fbf80-e83c-11ea-9b56-a068ea3155c6.png
:alt: my-picture1
:align: center

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add the caption:
(Privacy Preserving AI by Andrew Trask | MIT Deep Learning Series )

Copy link

@TravellerOnCode TravellerOnCode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested Changes:

  • Update the caption for the Images
  • Under the section: DIFFERENTIAL - PRIVACY IN REAL WORLD, just add "OpenDp by Harvard University and Microsoft "

Rest looks good, feel free to make any changes in the layout as per your choice.

@8bitmp3
Copy link
Contributor

8bitmp3 commented Aug 28, 2020

Awesome PR @shivaylamba! Are you open to edits to the text? There are a few things that stand out that could be polished.

@chinmayshah99
Copy link
Member

chinmayshah99 commented Aug 28, 2020

@8bitmp3 please go ahead with the suggestions! You can review the PR

@shivaylamba
Copy link
Contributor Author

Awesome PR @shivaylamba! Are you open to edits to the text? There are a few things that stand out that could be polished.

Yes @8bitmp3 Please let me know regarding any changes that I should make!

@shivaylamba
Copy link
Contributor Author

@TravellerOnCode I have made the suggested changes. Also I tried to centrally align the images but it always defaults to aligning to the left. I had a look at this open issue from 7 years back : github/markup#163

@chinmayshah99 chinmayshah99 merged commit c654bb9 into OpenMined:dev Aug 29, 2020
@chinmayshah99 chinmayshah99 removed the Status: Review Needed 🙋 This needs someone to approve, deny, comment, or request changes label Aug 29, 2020
@chinmayshah99 chinmayshah99 linked an issue Sep 4, 2020 that may be closed by this pull request
@chinmayshah99
Copy link
Member

Fixes #207

dvadym pushed a commit to dvadym/PyDP that referenced this pull request Jul 3, 2022
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.

Introduction to DP
5 participants