Move pydp folder under src

OpenMined / PyDP

The Python Differential Privacy Library. Built on top of: https://github.com/google/differential-privacy

Apache License 2.0

507 stars 138 forks source link

Move pydp folder under src #291

Closed alejandrosame closed 3 years ago

alejandrosame commented 4 years ago

What?

Python code now lives under ./pydp, leaving our source code distributed over 2 folders.

Why?

All source code should be under ./src folder. In order to have a clean repository, we might need to restructure folders inside ./src and refactor the way the bindings are made. However, it shouldn’t be too much of a drastic change (mostly folder renaming, with its corresponding bazel configuration changes).

chinmayshah99 commented 4 years ago

Note: This would require playing with bazel configuration. Familiarity with bazel (or willingness to learn about bazel features) is a must.

siddhesh1598 commented 4 years ago

Can I work on this? I am not familiar with bazel, but I would like to learn and work on it.

chinmayshah99 commented 4 years ago

Sure, hit us on slack channel #lib_pydp if you need anything

siddhesh1598 commented 4 years ago

Alright, will do.

chinmayshah99 commented 4 years ago

hi @siddhesh1598, how is it coming along?

siddhesh1598 commented 4 years ago

Hey @chinmayshah99, so far so good. Started learning the bazel config and cloned the repo. Will soon make changes to it.

alejandrosame commented 3 years ago

Fixed on #332.