-
Notifications
You must be signed in to change notification settings - Fork 15
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
GPMLs that break json-svg conversion #12
Comments
I quickly checked WP91 and the only special thing I saw is the "^" character in some of the compound names. Could that reason? For WP435 Tyms seems to have two anchors. I don't expect that to be the problem, but it is probably wrong anyway. |
Manually uploaded old PNG and SVG to wp-assets and wp.gh.io/assets/img for these cases: WP435 |
Error for a new problematic GPML WP5271
|
@hasanbalci This ticket has a short list of GPMLs that break the json-svg step. While you're looking into fixes to include in the next release of pvjson, you might want to run each of these and see if you have any ideas. |
I had a chance to check what the problems are with these pathway files and here are my initial observations: WP435: I think this enters into an infinite loop somewhere during the generation of svg (in pvjs repo), because it gives memory error after some time. WP466, WP983 and WP3197: It seems there are some arcs and/or nodes in these pathways that pvjs repo doesn't like while trying to generate the corresponding svg images. I identified some of them and if I remove those graph elements temporarily, svgs are generated but this time svg-to-png conversion gives error and cannot generate png images. WP2371 and WP5287: It seems that svgs are generated but svg-to-png conversion gives error and cannot generate png images. |
WP590 also enters into infinite loop like WP435 while generating svg. |
Just to keep track of the remaining "problem children" from the recent sync of all approved pathways, here's a list pathways that can't be converted to pvjson, svg and png using our main script. I debugged quite a few, but these remain error-prone and I have no idea what to do about them. So, I'm ignore them for now :)
WP435
WP466
WP983
WP2371
WP3197
WP5287
WP590
The text was updated successfully, but these errors were encountered: