New backend endpoints for SDK [PD1-33] #2
Conversation
michal-chrobok-semantive
commented
Jun 23, 2025
michal-chrobok-semantive
commented
- Modified SDK to use new backend endpoints
- Removed Pandas
- Added Apache Arrow support
michal-chrobok-semantive
changed the title
PD1-33 IaC for AWS Greengrass Core deployment
Configure AWS GGC on a edge device, loaded with Ignition Edge, with full certificates, offline buffering, sending data to EMQX |
michal-chrobok-semantive
changed the title
There was a problem hiding this comment.
We should not mention "Quix" in the SDK examples. The client will reach our API through TENANT.cvector.app. In any case, Quix will be out of use within 1 month, since they are cancelling our hosting plan.
The client should not need to redundantly specify the tenant in both the subdomain and the custom header. Instead, we can use the router to enrich the API origin request with the Tenant ID. https://github.com/CVector-Energy/cvec-op-envs/pull/113
|
|
||
| def main() -> None: | ||
| cvec = CVec( | ||
| host="https://cvec-backend-rzhang-cvec-sandbox.deployments.quix.io", |
There was a problem hiding this comment.
| host="https://cvec-backend-rzhang-cvec-sandbox.deployments.quix.io", | |
| host="https://CUSTOMER.cvector.app", |
examples/add_metric_data_example.py
Outdated
|
|
||
| def main() -> None: | ||
| cvec = CVec( | ||
| host="https://cvec-backend-rzhang-cvec-sandbox.deployments.quix.io", |
There was a problem hiding this comment.
| host="https://cvec-backend-rzhang-cvec-sandbox.deployments.quix.io", | |
| host="https://CUSTOMER.cvector.app", |
examples/get_metric_arrow_example.py
Outdated
|
|
||
| def main() -> None: | ||
| cvec = CVec( | ||
| host="https://cvec-backend-rzhang-cvec-sandbox.deployments.quix.io", |
There was a problem hiding this comment.
| host="https://cvec-backend-rzhang-cvec-sandbox.deployments.quix.io", | |
| host="https://CUSTOMER.cvector.app", |
|
|
||
| def main() -> None: | ||
| cvec = CVec( | ||
| host="https://cvec-backend-rzhang-cvec-sandbox.deployments.quix.io", |
There was a problem hiding this comment.
| host="https://cvec-backend-rzhang-cvec-sandbox.deployments.quix.io", | |
| host="https://CUSTOMER.cvector.app", |
joshuanapoli
left a comment
joshuanapoli
left a comment
There was a problem hiding this comment.
We should update the SDK to assume that the Router will add the x-tenant-id header without involving the client. https://github.com/CVector-Energy/cvec-op-envs/pull/113
|
@joshuanapoli I think this PR is ready for re-review and merging |
