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

Signed ints are output as unsigned in ezpc.h #4

Open
nishkum opened this issue Jul 12, 2019 · 0 comments · May be fixed by #71
Open

Signed ints are output as unsigned in ezpc.h #4

nishkum opened this issue Jul 12, 2019 · 0 comments · May be fixed by #71
Labels
enhancement New feature or request

Comments

@nishkum
Copy link
Contributor

nishkum commented Jul 12, 2019

Today if signed ints are used in output statement, their unsigned versions are output.
For example,
def void main(){
int32_al x = -1;
output(SERVER, x);
}

The above function will print (1<<32)-1 and not -1.
Solution:
Fix ezpc.h so that output queue has output variable type in account.

@mayank0403 mayank0403 added the enhancement New feature or request label Sep 27, 2019
@rasswanth-s rasswanth-s linked a pull request Jan 10, 2021 that will close this issue
trajore pushed a commit to trajore/EzPC_1 that referenced this issue Dec 27, 2023
* Adding OnnxBridge and ezpc-cli from onnxbridge branch

* added code for image load, model weights load, with ip and party

* fixes

* minor fix

* output only to client

* function renamed bcuz of overloading

* final changes

* typo

* checkpoint Lenet working-> bug fixed

* script updated

* LLAMA compile script added

* LLAMA compile script update

* script working

* basic README added

* updated readme, need to still update about LLAMA prereq

* updated readme, need to still update about LLAMA prereq

* typo

* updated to automatically send zip file over http

* updates

* bugfix

---------

Co-authored-by: Kanav Gupta <[email protected]>
Co-authored-by: Kanav Gupta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants