Skip to content

Feat/add twinbuilder example #751

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

Merged
merged 11 commits into from
Jan 21, 2022
Merged

Feat/add twinbuilder example #751

merged 11 commits into from
Jan 21, 2022

Conversation

SushobhitGupt
Copy link
Contributor

@SushobhitGupt SushobhitGupt commented Jan 21, 2022

Add a new RC Circuit example for Twin Builder
Add a new Voltage Source Component for Twin Builder

@MaxJPRey
Copy link
Collaborator

@sushobhit-gupt
In each folder containing an example we have a Readme.txt file presenting the purpose of those examples. Maybe you can add it.
Otherwise, it looks good to me and very nice example.

@maxcapodi78
Copy link
Collaborator

@sushobhit-gupt @MaxJPRey I agree, we need the readme.txt in the folder. This will be used in the header of the examples in online documentation. You can take as reference one of the other readme files.

Copy link
Collaborator

@maxcapodi78 maxcapodi78 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the Readme.txt. Then it looks great to me.

@SushobhitGupt
Copy link
Contributor Author

SushobhitGupt commented Jan 21, 2022

Please add the Readme.txt. Then it looks great to me.

@maxcapodi78 @MaxJPRey I have added the new readme for Twin Builder and merged the changes from @MaxJPRey .
Please take a look when you get a chance.

@maxcapodi78 maxcapodi78 merged commit fbb2eea into main Jan 21, 2022
@maxcapodi78 maxcapodi78 deleted the feat/add-twinbuilder-example branch January 21, 2022 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants