Skip to content

Commit 4a10536

Browse files
Merge pull request #1120 from slactjohnson/nttable-npint
BUG: Add a case in p4p_plugin_component.py for Numpy integers.
2 parents 6cf1d8d + f196c2e commit 4a10536

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

pydm/data_plugins/epics_plugins/p4p_plugin_component.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -300,6 +300,8 @@ def send_new_value(self, value: Value) -> None:
300300
self.new_value_signal[str].emit(new_value)
301301
elif isinstance(new_value, dict):
302302
self.new_value_signal[dict].emit(new_value)
303+
elif isinstance(new_value, np.integer):
304+
self.new_value_signal[int].emit(int(new_value))
303305
else:
304306
raise ValueError(f"No matching signal for value: {new_value} with type: {type(new_value)}")
305307
# Sometimes unchanged control variables appear to be returned with value changes, so checking against

pydm/tests/data_plugins/test_p4p_plugin_component.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ def generate_control_variables(value):
5151
1,
5252
),
5353
(NTEnum().wrap({"index": 0, "choices": ["YES", "NO", "MAYBE"]}), False, 0, 2),
54-
],
54+
], # Add cases for testing NTTable containing np.integer and np float types arrays here
5555
)
5656
def test_send_new_value(
5757
monkeypatch: MonkeyPatch,

0 commit comments

Comments
 (0)