-
Notifications
You must be signed in to change notification settings - Fork 10.1k
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
Groups path commands into single operation #4683
Conversation
/botio-linux preview |
From: Bot.io (Linux)ReceivedCommand cmd_preview from @timvandermeij received. Current queue size: 0 Live output at: http://107.21.233.14:8877/26070e1f991b61e/output.txt |
From: Bot.io (Linux)SuccessFull output at http://107.21.233.14:8877/26070e1f991b61e/output.txt Total script time: 0.37 mins Published
|
The first PDF is goes down from approx. 12 seconds to 9 seconds, the second PDF goes down from approx. 25 seconds to 20 seconds. Rendering also looks more constant. Nice work! |
.. thinking about further grouping those into set of figures: color + path + stroke + fill group commands |
One more to test with:
Regressed:
|
Let's land this. Grouping by shape is getting complicates (see groupshapes branch) /botio test |
From: Bot.io (Windows)ReceivedCommand cmd_test from @yurydelendik received. Current queue size: 0 Live output at: http://107.22.172.223:8877/04715530fa4f316/output.txt |
From: Bot.io (Linux)ReceivedCommand cmd_test from @yurydelendik received. Current queue size: 0 Live output at: http://107.21.233.14:8877/9f2a212aa5c8b03/output.txt |
From: Bot.io (Windows)FailedFull output at http://107.22.172.223:8877/04715530fa4f316/output.txt Total script time: 22.08 mins
Image differences available at: http://107.22.172.223:8877/04715530fa4f316/reftest-analyzer.html#web=eq.log |
From: Bot.io (Linux)FailedFull output at http://107.21.233.14:8877/9f2a212aa5c8b03/output.txt Total script time: 25.79 mins
Image differences available at: http://107.21.233.14:8877/9f2a212aa5c8b03/reftest-analyzer.html#web=eq.log |
/botio-windows unittest |
From: Bot.io (Windows)ReceivedCommand cmd_unittest from @yurydelendik received. Current queue size: 0 Live output at: http://107.22.172.223:8877/b15bff83d3df5ec/output.txt |
From: Bot.io (Windows)SuccessFull output at http://107.22.172.223:8877/b15bff83d3df5ec/output.txt Total script time: 2.74 mins
|
/botio test |
From: Bot.io (Windows)ReceivedCommand cmd_test from @timvandermeij received. Current queue size: 0 Live output at: http://107.22.172.223:8877/4e45f71a2ce8277/output.txt |
From: Bot.io (Linux)ReceivedCommand cmd_test from @timvandermeij received. Current queue size: 0 Live output at: http://107.21.233.14:8877/9ce506fa882d782/output.txt |
From: Bot.io (Windows)FailedFull output at http://107.22.172.223:8877/4e45f71a2ce8277/output.txt Total script time: 22.01 mins
Image differences available at: http://107.22.172.223:8877/4e45f71a2ce8277/reftest-analyzer.html#web=eq.log |
From: Bot.io (Linux)FailedFull output at http://107.21.233.14:8877/9ce506fa882d782/output.txt Total script time: 25.79 mins
Image differences available at: http://107.21.233.14:8877/9ce506fa882d782/reftest-analyzer.html#web=eq.log |
Given that the test failures only affect Firefox, and the fact that the changes are really small, could this by any change be caused by a Firefox update on the bots? |
Let's test this again now that the latest references have been made. /botio test |
From: Bot.io (Linux)ReceivedCommand cmd_test from @timvandermeij received. Current queue size: 0 Live output at: http://107.21.233.14:8877/7f8eb9e9d7e7346/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @timvandermeij received. Current queue size: 0 Live output at: http://107.22.172.223:8877/b330433c3dab050/output.txt |
From: Bot.io (Windows)FailedFull output at http://107.22.172.223:8877/b330433c3dab050/output.txt Total script time: 22.41 mins
Image differences available at: http://107.22.172.223:8877/b330433c3dab050/reftest-analyzer.html#web=eq.log |
From: Bot.io (Linux)FailedFull output at http://107.21.233.14:8877/7f8eb9e9d7e7346/output.txt Total script time: 25.79 mins
Image differences available at: http://107.21.233.14:8877/7f8eb9e9d7e7346/reftest-analyzer.html#web=eq.log |
/botio-windows preview |
From: Bot.io (Windows)ReceivedCommand cmd_preview from @yurydelendik received. Current queue size: 0 Live output at: http://107.22.172.223:8877/17e040f29cfb281/output.txt |
From: Bot.io (Windows)SuccessFull output at http://107.22.172.223:8877/17e040f29cfb281/output.txt Total script time: 0.89 mins Published
|
/botio test |
From: Bot.io (Linux)ReceivedCommand cmd_test from @yurydelendik received. Current queue size: 0 Live output at: http://107.21.233.14:8877/c3e0b80dd38683c/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @yurydelendik received. Current queue size: 0 Live output at: http://107.22.172.223:8877/50b401eec0ef650/output.txt |
From: Bot.io (Windows)FailedFull output at http://107.22.172.223:8877/50b401eec0ef650/output.txt Total script time: 23.74 mins
Image differences available at: http://107.22.172.223:8877/50b401eec0ef650/reftest-analyzer.html#web=eq.log |
From: Bot.io (Linux)FailedFull output at http://107.21.233.14:8877/c3e0b80dd38683c/output.txt Total script time: 25.58 mins
Image differences available at: http://107.21.233.14:8877/c3e0b80dd38683c/reftest-analyzer.html#web=eq.log |
Okay, looks like commands are split in chunks different way, so small images chunk optimization produce different results. I think we good to go. /botio makeref |
From: Bot.io (Windows)ReceivedCommand cmd_makeref from @yurydelendik received. Current queue size: 0 Live output at: http://107.22.172.223:8877/5ebec9fee16f1e8/output.txt |
From: Bot.io (Linux)ReceivedCommand cmd_makeref from @yurydelendik received. Current queue size: 0 Live output at: http://107.21.233.14:8877/f787c6df6047813/output.txt |
From: Bot.io (Windows)SuccessFull output at http://107.22.172.223:8877/5ebec9fee16f1e8/output.txt Total script time: 23.90 mins
|
From: Bot.io (Linux)SuccessFull output at http://107.21.233.14:8877/f787c6df6047813/output.txt Total script time: 25.67 mins
|
Groups path commands into single operation
No description provided.