You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@teaclave.apache.org by Mingshen Sun <no...@github.com.INVALID> on 2021/12/14 23:35:50 UTC
[apache/incubator-teaclave] Refactor Python client SDK (PR #591)
## Description
Refactor Python client SDK. There are several changes:
- Add TeacalveException, raise when needed
- Add close channel function and support context manager functions (i.e., `with xxx as:`)
- Remove client classes and merge all functions into the service classes
- Add a parent class (`TeaclaveService`) to merge common APIs in service class
## Type of change (select or add applied and delete the others)
- [x] New feature (non-breaking change which adds functionality)
- [x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [x] API change with a documentation update
## How has this been tested?
CI tests.
## Checklist
- [x] Fork the repo and create your branch from `master`.
- [x] If you've added code that should be tested, add tests.
- [x] If you've changed APIs, update the documentation.
- [x] Ensure the tests pass (see CI results).
- [x] Make sure your code lints/format.
You can view, comment on, or merge this pull request online at:
https://github.com/apache/incubator-teaclave/pull/591
-- Commit Summary --
* Refactor python client SDK
-- File Changes --
M examples/python/builtin_echo.py (64)
M examples/python/builtin_face_detection.py (21)
M examples/python/builtin_gbdt_train.py (22)
M examples/python/builtin_online_decrypt.py (20)
M examples/python/builtin_ordered_set_intersect.py (28)
M examples/python/builtin_password_check.py (27)
M examples/python/builtin_private_join_and_compute.py (43)
M examples/python/builtin_rsa_sign.py (30)
M examples/python/mesapy_echo.py (21)
M examples/python/mesapy_logistic_reg.py (20)
M examples/python/utils.py (20)
M examples/python/wasm_c_simple_add.py (22)
M examples/python/wasm_rust_psi.py (26)
M examples/python/wasm_tvm_mnist.py (28)
M sdk/python/teaclave.py (270)
-- Patch Links --
https://github.com/apache/incubator-teaclave/pull/591.patch
https://github.com/apache/incubator-teaclave/pull/591.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-teaclave/pull/591
Re: [apache/incubator-teaclave] Refactor Python client SDK (PR #591)
Posted by Mingshen Sun <no...@github.com.INVALID>.
Merged #591 into master.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-teaclave/pull/591#event-5767970004