[GitHub] [guacamole-server] andrewsali commented on pull request #322: Force fast WebP compression

[GitBox <gi...@apache.org>]

andrewsali commented on pull request #322:
URL: https://github.com/apache/guacamole-server/pull/322#issuecomment-755181167


   Sorry, this PR is not meant to be a definitive code-base change at all - it is just a suggestion for testing and if others can reproduce the improved speed, then formalize the aspects requested in the code-review.
   
   For encoding speed / bandwidth, what we tested was to take a screenshot PNG (Full HD screen) and see how much time it is to convert to webp using the cwebp utility.
   
   Using m=0, q=30, converting 50 times
   
   ```
   real    0m2.549s
   user    0m2.151s
   sys     0m0.404s
   ```
   
   Output file size: 220K
   
   Using m=2, q=30, converting 50 times:
   ```
   real    0m13.037s
   user    0m12.575s
   sys     0m0.466s
   ```
   
   Output file size: 112K
   
   The original PNG file size was also 220K.
   
   Using m=0, q=90, converting 50 times:
   
   ```
   real    0m3.201s
   user    0m2.945s
   sys     0m0.261s
   ```
   
   Output file size: 186K.
   
   Using m=2, q=90, converting 50 times:
   
   ```
   real    0m15.867s
   user    0m15.531s
   sys     0m0.333s
   ```
   
   Output file size: 103K.
   
   So compared to the m=2 setting, the bandwidth requirement seems about double, but encode speeds are about x5.
   
   Given the improvements in bandwidth, it might be worthwhile to go for faster compression. In our subjective experience this has helped substantially. But it would be good for some tests in other deployments to see if it is really a good way to go.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org