Skip to content
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

Added to the Show command a -wireshape <graphviz-shape> flag. #4252

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

zapta
Copy link

@zapta zapta commented Mar 2, 2024

This PR adds the flag -wireshape <graphviz-shape> to the Show command and allows to control the shape of wire nodes in the generated graph. The default behavior of using a diamond shape stays the same. The motivation is to allow users to reduce the visual load of wires.

Before (also the new default behavior and with -wireshape diamond)
_before

After, with -wireshape plaintext.
_plaintext

Dot files
Archive.zip

This allows to control the shape of wire nodes, for example, -wireshape plaintext.
The motivation is to allow the user to reduce visual loads of wires.
This does not change the default behavior of using a diamond shape.
@zapta
Copy link
Author

zapta commented Mar 2, 2024

@povik, is this build failure related to my change?

...
[100%] Building yosys.js
emcc: error: --memory-init-file is no longer supported
make: *** [Makefile:738: yosys.js] Error 1
Error: Process completed with exit code 2.

@povik
Copy link
Member

povik commented Mar 2, 2024

I don't think so. Pinging @whitequark, maybe she will know right away.

@KrystalDelusion
Copy link
Member

Build failure is due to latest release of emscripten. #4255 fixes it :)

@zapta
Copy link
Author

zapta commented Mar 4, 2024

Will this be retried automatically now that #4255 is in?

@KrystalDelusion
Copy link
Member

KrystalDelusion commented Mar 4, 2024 via email

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