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
Adding GitHub Actions for LVS testing #12
Conversation
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.
Why are sram test cases being added under the sc_testscases
directory which I assume stands for "standard cell"?
@umarcor - I would appreciate your thoughts on this pull request. |
@mohanad0mohamed Please split the run to smaller runs allowing the github actions to be parallelized and controlled easier. @mithro You are right it stand for SC. But testing the RAM as well would be helpful. It's basically part of the digital library. |
be2e2bf
to
67c9e7c
Compare
@mithro @atorkmabrains All requested changes had been made. Note: the SRAM cells aren't included because they are still under development for LVS matching. |
67c9e7c
to
c2f02c6
Compare
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.
I'm going to merge this pull request, but follow on work should be done to address #13
Updating primitives names in [ip , manual, extraction checking] testc…
Fixes #4
All actions required for LVS are now running.