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

First-pass DP PR #43

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

First-pass DP PR #43

wants to merge 5 commits into from

Conversation

FishmanL
Copy link

@FishmanL FishmanL commented Apr 3, 2020

The code is relatively simple -- I just used the RGSL package for laplacian noise which I generate, take the absolute value of, double (since I'm only taking 1 tail) and then subtract it from the distance before comparing it to the minimum.

@lacabra lacabra self-requested a review April 8, 2020 21:32
@lacabra
Copy link
Contributor

lacabra commented Apr 8, 2020

Thanks @FishmanL, I've taken a look at your PR. I've made some small adjustments but this PR won't compile because the RGSL library depends on features from the std library which are not available inside SGX, and that one cannot disable.

@reuvenpo do you have any suggestions?

@FishmanL
Copy link
Author

FishmanL commented Apr 8, 2020

@lacabra
Copy link
Contributor

lacabra commented Apr 8, 2020

Possibly. That's outside my knowledge of SGX+Rust. That's why I tagged Reuven in my previous comment. I'll let him respond.

@assafmo
Copy link
Member

assafmo commented Apr 19, 2020

Hi @FishmanL, we can only use no_std-enabled crates inside the enclave, so please use a GSL library with no_std support and make sure the project compiles on your machine. 😊

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.

3 participants