Skip to content

🎉Server client#435

Draft
fbordeu wants to merge 12 commits into
mainfrom
ServerClient
Draft

🎉Server client#435
fbordeu wants to merge 12 commits into
mainfrom
ServerClient

Conversation

@fbordeu
Copy link
Copy Markdown
Collaborator

@fbordeu fbordeu commented Jun 3, 2026

Server to connect to a predict implementation.
This tool need only plaid installed locally

@fbordeu fbordeu requested a review from a team as a code owner June 3, 2026 07:03
@fbordeu fbordeu marked this pull request as draft June 3, 2026 07:03
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 3, 2026

Codecov Report

❌ Patch coverage is 98.02956% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/plaid/utils/sample_json.py 93.47% 3 Missing ⚠️
src/plaid/utils/cgns_json.py 98.83% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Comment thread examples/client_server/SimplePredict.py Fixed
Comment thread examples/client_server/SimplePredict.py Fixed
Comment thread examples/client_server/SimplePredict.py Fixed
Comment thread examples/client_server/SimplePredict.py Fixed
fbordeu and others added 4 commits June 3, 2026 17:04
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
@casenave
Copy link
Copy Markdown
Member

casenave commented Jun 3, 2026

Can you add docs entries to explain the usage ? Do you think the serializer can be used in the viewer to prevent the return trip to the disk ? Thanks

@fbordeu
Copy link
Copy Markdown
Collaborator Author

fbordeu commented Jun 3, 2026

The example SimplePredict shows a usage. But for the moment the server is not available on this repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants