Skip to content
This repository has been archived by the owner on Dec 10, 2017. It is now read-only.

Refactored capsule server and fixed bug #27

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

Conversation

cosmadrian
Copy link

This PR addresses issue #26.

Removed duplicate code for capsule_zmq and capsule_mpc, now both are operating from capsule_server.
The capsule_server/local_server.py script now accepts two flags, one for the type of tasks that it handles
(mpc or zmq) and the port.
Capsule server now sends pickled dictionary responses that contain a 'success' flag. When success is false,
an exception is thrown at the client side.
On the capsule client, exception handling for the new 'protocol' is done in an unobtrusive way, by means of
the TaskSocket class that wraps zmq.socket.
Changed build_and_run.sh script to reflect these changes.

Removed duplicate code for capsule_zmq and capsule_mpc, now both are operating from capsule_server.
The capsule_server/local_server.py script now accepts two flags, one for the type of tasks that it handles
(mpc or zmq) and the port.
Capsule server now sends pickled dictionary responses that contain a 'success' flag. When success is false,
an exception is thrown at the client side.
On the capsule client, exception handling for the new 'protocol' is done in an unobtrusive way, by means of
the TaskSocket class that wraps zmq.socket.
Changed build_and_run.sh script to reflect these changes.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant