doc: Rearrange a few lines in the dependency graph of libraries #26358

pull satsie wants to merge 1 commits into bitcoin:master from satsie:patch-1 changing 1 files +5 −5
  1. satsie commented at 2:30 AM on October 21, 2022: contributor

    In this PR, I've attempted to improve readability in the dependency graph of libraries by untangling a few crossed lines. I'm not sure if this is that big of an improvement but wanted to throw it out there.

    I used an extremely scientific method of manually counting the number of crossed lines in the original diagram and got 15. This PR reduces that number down to about 10.

    I also changed the curve of the lines to "basis" which rounds the edges out. Again, not sure if it really is that much of an improvement, but it seems marginally easier on the eyes.

    Here is what the new graph looks like rendered:

    Screenshot from 2022-10-20 22-09-30

    The changes can be verified independently with Mermaid, with the easiest way being the online editor: https://mermaid.live/

    I did try moving some more stuff around, particularly the top level of library callers, but was not able to simplify the graph any further.

  2. doc: Rearrange some lines in the dependency graph of libraries 1184a66347
  3. fanquake added the label Docs on Oct 21, 2022
  4. kevkevinpal approved
  5. kevkevinpal commented at 3:54 AM on October 21, 2022: contributor

    ACK looks good to me and is easier on the eyes

  6. shaavan approved
  7. shaavan commented at 7:45 AM on October 21, 2022: contributor

    ACK 1184a66347afbaeca69e2d891a138e435fbfd5bb

    The graph looks a lot cleaner now.

    Here's a screenshot of the difference between the original and the PR.

    Screenshot from 2022-10-21 12-55-34

    I was also able to verify the markdown code using mermaid.live independently

    I also tried to experiment with moving the boxes (in fact, I think I spent a little too much time on it :), but I could not further reduce the number of crosses in the graph.

  8. maflcko merged this on Oct 24, 2022
  9. maflcko closed this on Oct 24, 2022

  10. satsie deleted the branch on Oct 24, 2022
  11. sidhujag referenced this in commit 3cfe69e1ec on Oct 25, 2022
  12. bitcoin locked this on Oct 24, 2023
Labels

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-19 03:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me