Skip to content

Conversation

@DanGould
Copy link
Contributor

Updated sequence diagrams to use text format instead of mermaid syntax responding to @achow101's comment

I cargo cult'd the RFC Rules:

“How are images handled for the plain text version of an RFC?”
The RFC Editor will accept both ASCII art and SVG. If only ASCII art is provided, it will be included in all publication formats. If ASCII art and SVG are both provided, ASCII art will be included in the plain text, and SVG in all other outputs. A note indicating alternative artwork is available is strongly advised. If only SVG is provided, a URI will be included in the plain-text publication format pointing to the HTML version. All artwork and figures should have a complete written description to support assisted reader technology.

see: https://www.rfc-editor.org/rse/format-faq/

Since BIPs don't publish html/pdf renders, ASCII art seems like the right choice to render everywhere. Since normative prose is already provided, I chose not to include a written description of the diagrams to support assisted reader tech.

cc @nothingmuch

@DanGould DanGould mentioned this pull request Dec 18, 2025
Updated sequence diagrams to use text format instead of mermaid syntax.

I cargo cult'd the RFC Rules:

> “How are images handled for the plain text version of an RFC?”
> The RFC Editor will accept both ASCII art and SVG. If only ASCII art is provided, it will be included in all publication formats. If ASCII art and SVG are both provided, ASCII art will be included in the plain text, and SVG in all other outputs. A note indicating alternative artwork is available is strongly advised. If only SVG is provided, a URI will be included in the plain-text publication format pointing to the HTML version. All artwork and figures should have a complete written description to support assisted reader technology.

see: https://www.rfc-editor.org/rse/format-faq/

Since BIPs don't publish html/pdf renders, ASCII art seems like the right choice to render everywhere. Since normative prose is already provided, I chose not to include a written description of the diagrams to support assisted reader tech.
@ajtowns
Copy link
Contributor

ajtowns commented Dec 18, 2025

Rather than downgrading to ascii art, maybe include the mermaid source in a bip-0077 directory and generate an svg from it (either using mermaid-cli or copy/pasting into the live editor and exporting?) At the very least, mermaid source is easier to edit than ascii art.

@DanGould
Copy link
Contributor Author

This was something we bikeshedded even in the original PR and is not the most consequential. I am revisiting only because of a request. ascii art is about as simple as possible, and these diagrams are ancient at this point and really don't need to change. LLMs will produce ascii art from mermaid source, which is now in the commit history. I'd like to go ahead with this as-is for simplicity unless there's a really really good reason to change again.

@DanGould
Copy link
Contributor Author

DanGould commented Jan 9, 2026

I'm not sure when the unanimous agreement on @ajtowns 's suggestion came through via "👍" because I don't get notified by emojis. I would have addressed this earlier if someone reinforced that my prior approach was unacceptable by comment. Should be good to go now.

@DanGould DanGould force-pushed the patch-2 branch 2 times, most recently from 16d67a5 to 0bc0550 Compare January 9, 2026 05:33
@achow101
Copy link
Member

achow101 commented Jan 9, 2026

(non blocking comment)

I liked the ascii art better. The svg is not sized correctly and doesn't have a transparent background for those of us using dark mode.

Also I can't figure out which settings I had turned on that caused the rendering issue in the first place.

@DanGould
Copy link
Contributor Author

DanGould commented Jan 9, 2026

sizing is fixable in github with progressive css (is that really what we want here) but transparent background likely won't work because I'd have to use custom theme to render the svg with diagram lines that look ok on every possible background.

This is major bikeshedding. ASCII "just works" all the time and you can really tell a free LLM "add this line" "remove x part" these days. editing ascii art is a non-issue in 2026.

@DanGould
Copy link
Contributor Author

DanGould commented Jan 9, 2026

Ok after discussing with Ava I pushed the mermaid stuff here if you really want to look, but I promise it's a bigger pain in the hiney https://github.com/DanGould/bips/commits/mermaid-bip77/

I guess everyone just checked out for the holidays like myself. I'm re force-pushing ascii art to the pr and would appreciate a simple go-ahead as the original author editing his own bip with identical content in a different format. Thank you for your attention to this matter.

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