PR to pull back changes of IIC branch #3
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mkkassem
approved these changes
May 3, 2022
kareefardi
reviewed
May 4, 2022
images/iic-osic-tools/info.json
Outdated
| "NAME": "foss-asic-tools", | ||
| "REPO_URL": "https://github.com/efabless/foss-asic-tools", | ||
| "NAME": "iic-osic-tools", | ||
| "REPO_URL": "https://github.com/hpretl/iic-osic-tools", |
There was a problem hiding this comment.
this is just a reminder comment to change the url after merging
…t work, for whatever weird reason
This reverts commit ee2220b.
…iately after exiting. For mac, only allowing access from localhost is enough for Docker Desktop X-forwarding to work.
…rsion. Let's see how to get it to compile again.
…or LibreLane does not build anylonger
… build fail We can temporarily install a new `spdlog`, as it is not linked in the binary, just the header files are used.
…ld binding Ctrl+P is now used by the klayout-pin-tool
Klayout keybinding for adjust origin (as used by via tool) and creating polygons changed to Ctrl+Shift+P as old binding Ctrl+P is now used by the klayout-pin-tool
Fix pulp-flow test
Included libftdi-dev in the list of packages installed by the base image script to support FTDI device development.
klayout keybindings of DRC updated to use newest ihp DRC script
Replaced all mentions of 'DIC' (Department for Integrated Circuits) with 'ICD' (Department for Integrated Circuits) in documentation, build scripts, example schematics, and tool scripts to reflect the department's updated abbreviation. This change ensures consistency across the project and aligns with current institutional naming.
Updated eda_server_stop.sh to use 'docker ps -a' instead of 'docker ps' so that both running and stopped containers with the specified prefix are found and removed.
Updated start_vnc.bat and start_vnc.sh to omit the '--security-opt seccomp=unconfined' Docker parameter when the IIC_SERVER_DEPLOYMENT environment variable is set. This allows for more flexible deployment configurations.
Add python formatter black
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please cherry-pick which changes you want/need, as we changed quite a bit.
Our changes rebased CentOS7 to RockyLinux8, and now support
x86_64as well asaarch64. Plus, we cleaned up the build scripts, and added a few more tools.By default, only
sky130Ais installed along the_hdstandard cells.