TA save SVG saves nothing

[tonyforster]

The save SVG block is giving me nothing in the Journal
There is no error log
Save picture is OK

the turtle is hidden till you click stop

I vaguely recollect something like this before, SVG being procedural rather than the finished image, maybe I am doing something wrong? Adding a Start block didnt help.

[walter]

I am guessing that you were using fill blocks? There is a bug in the svg code for polygon fill (it wasn't updated to handle a syntax change when I added support for filled arcs). I've got a partial solution in [1]. Still need to add arc support.

[1] ​http://git.sugarlabs.org/turtleart/mainline/commit/07947d937cc274224618fbd6a94e0e6715f29124

[walter]

In general, the patch [2] seems to work. Further testing requested.

[2] ​http://git.sugarlabs.org/turtleart/mainline/commit/ac5389883626ca0b33ed8bb109fa39b588086b38

[tonyforster]

Applied both patches
able to save filled square OK
save svg only appears to save if it is docked to the blocks that are doing the drawing, if you are single stepping, nothing is saved. This is not easily discoverable.

SVG of filled arc does not work well, drawing is corrupted

[walter]

I did a refactoring... using a cairo surface now. Seems to have solved the SVG problems. Fixed this issue too, by checking to see if an SVG block is in use, even if it is not in the current stack.

​http://git.sugarlabs.org/turtleart/mainline/commit/f00e3fe2182c0bab2b802a7a988a3e0a286c8db5

[tonyforster]

works for me
filled arc 360 ok
filled arc 90 ok
single stepped filled square ok
rotated text ok

does not record in single step if the save svg block is introduced to the canvas after the drawing is done. This is presumably a compromise to save execution time, not keeping an unnecessary record of draw events in svg format?

[walter]

Thanks for testing.

does not record in single step if the save svg block is introduced to the canvas after the
drawing is done. This is presumably a compromise to save execution time, not keeping an
unnecessary record of draw events in svg format?

Yes. I only keep the SVG surface around if I think the user is intending to save to SVG at some point. I only make that assumption if a save SVG block is on the canvas somewhere. Otherwise, twice as much writing. I could revisit this assumption if you think it is worth the tradeoff (need to run some numbers on an XO 1 to measure the impact).

[tonyforster]

I agree, its not worth a noticable loss in execution speed.

[tonyforster]

xo-1.75 no measurable speed difference
blocks hidden (fastest mode)

repeat 2000 fwd100 rt90
print time

takes 9 seconds with or without a save svg on the canvas

[walter]

Need to do further testing on the impact on memory consumption as well, but in general, I think this would be a good thing to do, since it would also mean we could add a save as svg button to the toolbar.

[walter]

Fixed by [1]. Will be part of v185 release.

[1] ​https://git.sugarlabs.org/turtleart/mainline/commit/12eab0ca4c4eb13312cd9ec8b601528f6e9d07b8