Skip to content

Fixing server utilities#229

Merged
cplett merged 3 commits intofaccts:mainfrom
cplett:oet_fix
Apr 15, 2026
Merged

Fixing server utilities#229
cplett merged 3 commits intofaccts:mainfrom
cplett:oet_fix

Conversation

@cplett
Copy link
Copy Markdown
Contributor

@cplett cplett commented Apr 15, 2026

Description

  • Adjusting the server functionality to the recent changes of the oet. Previously, the server was always started with the python version used to execute OPI, which can lead to import errors when calling the oet scripts that were installed to a different environment.

Release Notes

Fixed

  • Adjusted the server functionality to work with recent changes in the orca-external-tools infrastructure.

Signed-off-by: Christoph Plett <plett@faccts.de>
@cplett cplett requested a review from a team as a code owner April 15, 2026 12:18
@cplett cplett added the bug Something isn't working label Apr 15, 2026
Copy link
Copy Markdown
Contributor

@timmyte timmyte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this. Please consider my remarks before merging. And also rerun Nox again.

Comment thread src/opi/external_methods/server.py
Comment thread examples/exmp046_server/job.py Outdated
Comment thread src/opi/external_methods/server.py
Comment thread src/opi/external_methods/server.py Outdated
cplett added 2 commits April 15, 2026 15:01
Signed-off-by: Christoph Plett <plett@faccts.de>
Signed-off-by: Christoph Plett <plett@faccts.de>
@cplett cplett merged commit c53543d into faccts:main Apr 15, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants