- Notifications
You must be signed in to change notification settings - Fork441
Change InputOutputSystem repr to show type, name, inputs, and outputs#699
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
coveralls commentedJan 29, 2022
I agree that a change would be useful here. More generally, it would be nice to think through what we want to have displayed in various situations:
Finally, PR#710 has a conflicting change since I reset therepr format there to include the type and system name. So an
I like the version in this draft PR better, but note that if you get something with lots of inputs and outputs, it could be a bit odd looking. |
PR#721 has a portion of this ( |
henklaak commentedFeb 20, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
This has been implemented since v0.10.0. |
Uh oh!
There was an error while loading.Please reload this page.
I didn't know about
InputOutputSystem
's__str__
implementation, and I find the__repr__
a bit bare-bones. With this change:This is a draft for comment - if it looks OK, I'll (eventually) improve it and add tests. One improvement I can see already: make the input and output lists a little prettier by removing the quotes.