Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for backwards compatibility, new feature ROIs and RODs for AutoFocus and new timeout mechanic for all clients #34

Merged
merged 18 commits into from
Apr 4, 2024

Conversation

Galoshi
Copy link
Collaborator

@Galoshi Galoshi commented Mar 26, 2024

No description provided.

tests/config.py Show resolved Hide resolved
source/rpc/imager.py Outdated Show resolved Hide resolved
source/rpc/imager.py Outdated Show resolved Hide resolved
source/rpc/imager.py Show resolved Hide resolved
source/rpc/imager.py Show resolved Hide resolved
source/pcic/utils.py Outdated Show resolved Hide resolved
source/rpc/application.py Show resolved Hide resolved
source/rpc/application.py Show resolved Hide resolved
source/rpc/proxy.py Outdated Show resolved Hide resolved
source/rpc/proxy.py Outdated Show resolved Hide resolved
Galoshi added 5 commits March 27, 2024 09:32
@Galoshi Galoshi requested a review from bnozka March 27, 2024 08:48
@Galoshi Galoshi linked an issue Mar 27, 2024 that may be closed by this pull request
tests/test_rpc.py Show resolved Hide resolved
tests/test_rpc.py Show resolved Hide resolved
source/rpc/proxy.py Outdated Show resolved Hide resolved
source/rpc/proxy.py Outdated Show resolved Hide resolved
source/rpc/client.py Outdated Show resolved Hide resolved
source/rpc/application.py Show resolved Hide resolved
source/device/client.py Outdated Show resolved Hide resolved
self._autoconnect = autoconnect
self._pcic = None
self._rpc = None
if autoconnect:
Copy link
Member

Choose a reason for hiding this comment

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

Here I would also favor the removal of the if statement. The RPC client does not connect by itself anyway, and the PCIC client handles the autoconnect itself (the flag is passed to it). Having both clients instantiated unconditionally this simplifies the later access, i.e. the added checks in lines 58-59 and 64-66 can be removed again.

source/device/client.py Show resolved Hide resolved
…removed unnecessary autoconnect methods and added tests for timeout mechanic for PCIC and RPC client
@Galoshi Galoshi requested a review from bnozka April 4, 2024 07:24
@Galoshi Galoshi requested a review from cfreundl April 4, 2024 07:24
@Galoshi Galoshi merged commit bc84fd3 into master Apr 4, 2024
3 checks passed
@Galoshi Galoshi deleted the rpc_client_imports branch April 4, 2024 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
3 participants