-
Notifications
You must be signed in to change notification settings - Fork 120
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Doc/tb example 2025r1 #5226
base: main
Are you sure you want to change the base?
Doc/tb example 2025r1 #5226
Conversation
Thanks for opening a Pull Request. If you want to perform a review write a comment saying: @ansys-reviewer-bot review |
] | ||
]) | ||
|
||
# Add Datapairs component pin so that its location can directly be retrieved #TODO - how to fix this as part of CircuitComponent/pins class |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Samuelopez-ansys I would appreciate some review on that part, as I needed to explicitly add the list of pins to get their position through 'location' and use for the create_wire function below. In other words, I would expect the ...pins[xxx].location to work on any component added on the schematic but for some reasons the logic does not work for the Datapairs block added, hence I needed to create/add the list of pins explicitly (similarly to what is done in the CircuitComponent class)
This PR adds a new Twin Builder example showing the creation of LTI ROM, simulation of the model and post processing of the results
It also performs a few small updates on existing examples