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

YosysHQ / yosys

Yosys Open SYnthesis Suite

https://yosyshq.net/yosys/

ISC License

3.31k stars 860 forks source link

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

Open zapta opened 4 months ago

zapta commented 4 months ago

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

zapta commented 4 months ago

@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 commented 4 months ago

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

KrystalDelusion commented 4 months ago

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

zapta commented 4 months ago

Will this be retried automatically now that https://github.com/YosysHQ/yosys/pull/4255 is in?

KrystalDelusion commented 4 months ago

You’ll need to rebase it on latest master to get the fix.