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&#39;ve added code that should be tested, add tests.
- [x] If you&#39;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