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.

Sign up for GitHub

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

Jump to bottom

Swap chart renderer to cloud run based gv renderer #12

Merged

grevian merged 2 commits into master from gv-renderer

Apr 10, 2020

Owner

grevian commented Apr 8, 2020 •

edited

Loading

I finally got around to standing up my own renderer, see https://github.com/grevian/graphviz-renderer-web for details, so target it with the sites renderer

This is deployed to http://clrun-dot-grevian-graphviz.appspot.com/graph right now and can be tested there

TODO

Test all examples
Expose new supported graph types
Expose better error messages


          Swap chart renderer to cloud run based gv renderer

da0d20e

grevian self-assigned this

grevian mentioned this pull request

Example #5 doesn't render as expected #13

Open


          Expose new graph types, show returned error messages

f79055b

Owner Author

grevian commented Apr 10, 2020

Test all examples
Ended up discovering Example #5 doesn't render as expected #13 and Example #6 renders inconsistently #14, but still proceeding with this PR
Expose new supported graph types
Expose better error messages
Both done in f79055b

grevian merged commit b1ce7d3 into master

grevian mentioned this pull request

Fix merge conflicts in: Use image charts #11

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet