Skip to content

Commit

Permalink
Old tango servers don't support the pipe API when used with PyTango v…
Browse files Browse the repository at this point in the history
…ersions less than 9. Since we don't support pipes yet anyway, we should handle this exception and move on.
  • Loading branch information
burkeds committed Nov 21, 2024
1 parent e4b18e5 commit 6ffc8fc
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions src/ophyd_async/tango/signal/_signal.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
SignalW,
SignalX,
)
from tango import AttrDataFormat, AttrWriteType, CmdArgType, DeviceProxy, DevState
from tango import AttrDataFormat, AttrWriteType, CmdArgType, DeviceProxy, DevState, NonSupportedFeature
from tango.asyncio import DeviceProxy as AsyncDeviceProxy

from ._tango_transport import TangoSignalBackend, get_python_type
Expand Down Expand Up @@ -174,8 +174,12 @@ async def infer_signal_type(
else:
dev_proxy = proxy

if tr_name in dev_proxy.get_pipe_list():
raise NotImplementedError("Pipes are not supported")
try:
if tr_name in dev_proxy.get_pipe_list():
raise NotImplementedError("Pipes are not supported")
except NonSupportedFeature:
pass


if tr_name not in dev_proxy.get_attribute_list():
if tr_name not in dev_proxy.get_command_list():
Expand Down

0 comments on commit 6ffc8fc

Please sign in to comment.