Skip to content

🎉Server client#435

Draft
fbordeu wants to merge 20 commits into
mainfrom
ServerClient
Draft

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

Conversation

@fbordeu

@fbordeu fbordeu commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

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

codecov Bot commented Jun 3, 2026

Copy link
Copy Markdown

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
@casenave

casenave commented Jun 3, 2026

Copy link
Copy Markdown
Member

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

fbordeu commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator Author

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

Comment thread src/plaid/cli/paraview_plugin/__init__.py Fixed
Comment thread src/plaid/cli/paraview_plugin/__init__.py Fixed
Comment thread src/plaid/cli/paraview_plugin/__init__.py Fixed
Comment thread src/plaid/cli/paraview_plugin/PlaidParaViewPlugin.py Fixed
Comment thread src/plaid/cli/paraview_plugin/PlaidParaViewPlugin.py Fixed
Comment thread src/plaid/cli/paraview_plugin/PlaidParaViewPlugin.py Fixed
Comment thread src/plaid/cli/paraview_plugin/PlaidParaViewPlugin.py Fixed
Comment thread ParaViewPlugin/PlaidPlugin.py Fixed
Comment thread ParaViewPlugin/PlaidPlugin.py Fixed
Comment thread ParaViewPlugin/PlaidPlugin.py Fixed
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Comment thread src/plaid/cli/paraview_plugin/PlaidParaViewPlugin.py Fixed
Comment thread ParaViewPlugin/PlaidPlugin.py Fixed
Comment thread ParaViewPlugin/PlaidPlugin.py Fixed
fbordeu and others added 4 commits June 10, 2026 10:00
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
…eption''

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Comment thread src/plaid/cli/paraview_plugin/PlaidParaViewPlugin.py Fixed
Comment thread src/plaid/cli/paraview_plugin/PlaidParaViewPlugin.py Fixed
fbordeu and others added 2 commits June 10, 2026 10:07
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
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