mirror of
https://github.com/bec-project/bec_widgets.git
synced 2025-07-14 03:31:50 +02:00
fix(rpc-base): deprecate widget_name in favor of object_name; closes #499
This commit is contained in:
@ -144,7 +144,7 @@ class RPCBase:
|
|||||||
def __repr__(self):
|
def __repr__(self):
|
||||||
type_ = type(self)
|
type_ = type(self)
|
||||||
qualname = type_.__qualname__
|
qualname = type_.__qualname__
|
||||||
return f"<{qualname} with name: {self.widget_name}>"
|
return f"<{qualname} with name: {self.object_name}>"
|
||||||
|
|
||||||
def remove(self):
|
def remove(self):
|
||||||
"""
|
"""
|
||||||
@ -152,13 +152,6 @@ class RPCBase:
|
|||||||
"""
|
"""
|
||||||
self._run_rpc("remove")
|
self._run_rpc("remove")
|
||||||
|
|
||||||
@property
|
|
||||||
def widget_name(self):
|
|
||||||
"""
|
|
||||||
Get the widget name.
|
|
||||||
"""
|
|
||||||
return self.object_name
|
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def _root(self) -> BECGuiClient:
|
def _root(self) -> BECGuiClient:
|
||||||
"""
|
"""
|
||||||
|
@ -148,13 +148,6 @@ def test_async_plotting(qtbot, bec_client_lib, connected_client_gui_obj):
|
|||||||
y_data, last_scan_data.devices.waveform.get("waveform_waveform", {}).read().get("value", [])
|
y_data, last_scan_data.devices.waveform.get("waveform_waveform", {}).read().get("value", [])
|
||||||
)
|
)
|
||||||
|
|
||||||
# Check displayed data
|
|
||||||
x_data_display, y_data_display = curve._get_displayed_data()
|
|
||||||
# Should be not more than 1% difference, actually be closer but this might be flaky
|
|
||||||
assert np.isclose(x_data_display[-1], x_data[-1], rtol=0.01)
|
|
||||||
# Downsampled data should be smaller than original data
|
|
||||||
assert len(y_data_display) < len(y_data)
|
|
||||||
|
|
||||||
|
|
||||||
def test_rpc_image(qtbot, bec_client_lib, connected_client_gui_obj):
|
def test_rpc_image(qtbot, bec_client_lib, connected_client_gui_obj):
|
||||||
gui = connected_client_gui_obj
|
gui = connected_client_gui_obj
|
||||||
|
@ -14,7 +14,7 @@ def test_rpc_base(rpc_base):
|
|||||||
ref = RPCReference(registry, rpc_base._gui_id)
|
ref = RPCReference(registry, rpc_base._gui_id)
|
||||||
|
|
||||||
assert ref._gui_id == rpc_base._gui_id
|
assert ref._gui_id == rpc_base._gui_id
|
||||||
assert ref.widget_name == rpc_base.widget_name
|
assert ref.object_name == rpc_base.object_name
|
||||||
assert ref.__str__() == rpc_base.__str__()
|
assert ref.__str__() == rpc_base.__str__()
|
||||||
assert ref.__repr__() == rpc_base.__repr__()
|
assert ref.__repr__() == rpc_base.__repr__()
|
||||||
|
|
||||||
@ -25,4 +25,4 @@ def test_rpc_base(rpc_base):
|
|||||||
assert ref.__repr__() == f"<Deleted widget with gui_id {rpc_base._gui_id}>"
|
assert ref.__repr__() == f"<Deleted widget with gui_id {rpc_base._gui_id}>"
|
||||||
|
|
||||||
with pytest.raises(DeletedWidgetError):
|
with pytest.raises(DeletedWidgetError):
|
||||||
ref.widget_name # Object no longer referenced in registry
|
ref._root # Object no longer referenced in registry
|
||||||
|
Reference in New Issue
Block a user