-
Notifications
You must be signed in to change notification settings - Fork 9.9k
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
Scale stroking line width when using a tiling pattern. #10850
Conversation
/botio test |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @brendandahl received. Current queue size: 0 Live output at: http://54.67.70.0:8877/e5ad45c3de00976/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @brendandahl received. Current queue size: 0 Live output at: http://54.215.176.217:8877/3fd12efeb995681/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.67.70.0:8877/e5ad45c3de00976/output.txt Total script time: 17.82 mins
|
From: Bot.io (Windows)SuccessFull output at http://54.215.176.217:8877/3fd12efeb995681/output.txt Total script time: 25.79 mins
|
Don't merge yet...I see another potential issue that I can fix that I want to bundle with this. |
This fixes #10566, which is probably the issue you were looking for. |
I think this is good to merge once the two comments above are resolved. |
7436405
to
446efab
Compare
I won't have time to do the full fix yet, so for now we can live with this workaround. I've filed #10955 for the full fix. |
/botio-linux preview |
From: Bot.io (Linux m4)ReceivedCommand cmd_preview from @timvandermeij received. Current queue size: 0 Live output at: http://54.67.70.0:8877/68c920d3d334c8f/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.67.70.0:8877/68c920d3d334c8f/output.txt Total script time: 1.70 mins Published |
Thank you for improving this!
Fixing this incrementally is perfectly fine because at least now it works and we have reference test coverage, making a future improvement easier to verify. (I'll make new reference images in the other PR.) |
Fixes #10566.