Skip to content

Add missing labels when returning TimeResponseData #892

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.

Sign up for GitHub

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

Merged
murrayrm merged 5 commits into from
May 27, 2023
Merged

Add missing labels when returning TimeResponseData #892

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
34a7a49
Add missing labels when returning TimeResponseData
joaoantoniocardoso May 25, 2023
a78e85c
Test if labels are transferred to the response and we can still relab…
joaoantoniocardoso May 20, 2023
56c72c7
use *_labels in input_output_response for consistency
murrayrm May 27, 2023
d744a54
Propagate labels and names when converting between mimo/siso/simo
joaoantoniocardoso May 27, 2023
bd83e8c
Fix _process_labels to accept a single str label
joaoantoniocardoso May 27, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix _process_labels to accept a single str label
  • Loading branch information
@joaoantoniocardoso
joaoantoniocardoso committed May 27, 2023
commit bd83e8c9060f69d9e87afd861dda4b4bf3223bd7
5 changes: 4 additions & 1 deletion control/timeresp.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -694,7 +694,10 @@ def _process_labels(labels, signal, length):
raise ValueError("Name dictionary for %s is incomplete" % signal)

# Convert labels to a list
labels = list(labels)
if isinstance(labels, str):
labels = [labels]
else:
labels = list(labels)

# Make sure the signal list is the right length and type
if len(labels) != length:
Expand Down