Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Python 2.7 support #51

Open
simra opened this issue May 29, 2020 · 2 comments
Open

Python 2.7 support #51

simra opened this issue May 29, 2020 · 2 comments
Labels
bug Something isn't working documentation Improvements or additions to documentation onboarding
Milestone
Documentation syn...

Comments

@simra
Copy link

simra commented May 29, 2020

Hi, a small suggestion to add a line to the DPSU spark implementation, to support down-level python:
from __future__ import division

https://github.com/opendifferentialprivacy/whitenoise-samples/blob/332d7ef7c4fcf811995f9e6a3aab8d8f4c298a88/data/policy_laplace.py#L1

Without the change, initializing PolicyLaplace incorrectly applies integer division in a few spots.
@raprasad raprasad added this to the Documentation sync/cleanup milestone Apr 14, 2021
@raprasad
Copy link
Member

Thanks*, @simra
(Much delayed)

@raprasad raprasad added bug Something isn't working documentation Improvements or additions to documentation labels Apr 14, 2021
@joshua-oss
Copy link
Contributor

Completely missed this. Great suggestion; I have patched it in #95

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working documentation Improvements or additions to documentation onboarding
Projects
None yet
Milestone
Documentation sync/cleanup
Development

No branches or pull requests
3 participants
@raprasad @simra @joshua-oss