Install GPU.js v2.3.1

[harshkhandeparkar]

Fixes #1348 (<=== Replace 0000 with the Issue Number
Fixes #1292

NOTE: This is a continuation of #1356

• Installed GPU.js v2.3.1 which fixes Blur module on small screen sizes
• @robertleeplummerjr the browserify error for gpu.js

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

• tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with npm test
• code is in uniquely-named feature branch and has no merge conflicts
• PR is descriptively titled
• ask @publiclab/is-reviewers for help, in a comment below
• Insert-step functionality is working correct as expected.

We're happy to help you get this ready -- don't be afraid to ask for help, and don't be discouraged if your tests fail at first!

If tests do fail, click on the red X to learn why by reading the logs.

Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software
Please make sure to get at least two reviews before asking for merging the PR as that would make the PR more reliable on our part
Thanks!

[harshkhandeparkar]

Other Changes:

• Text Overlay and Blob Analysis module tests failed due to the absence of WebGL in node. They also don't use puppeteer so they will not be tested. An issue regarding this will be opened soon.

[keshav234156]

https://travis-ci.org/publiclab/image-sequencer/jobs/627365619#L1479

[keshav234156]

test benchmarks are running now .Great!!!

[harshkhandeparkar]

Have a look at #1375

[harshkhandeparkar]

Also:

1. More modules eg: FisheyeGL and WebGLDistort which use puppeteer are now benchmarked.

[harshkhandeparkar]

Travis still fails though. I will find a solution for that.

[harshkhandeparkar]

@jywarren @keshav234156 @publiclab/is-reviewers This is working - at least locally. With the help of @robertleeplummerjr, I was able to successfully browserify the tests and run them. v2.3.1 of gpu.js and the later versions can be used now. The Blur module bug on smaller screens has been fixed now.

[codecov]

Codecov Report

Merging #1374 into main will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             main    #1374   +/-   ##
=======================================
  Coverage   66.12%   66.12%           
=======================================
  Files         125      125           
  Lines        2571     2571           
  Branches      404      404           
=======================================
  Hits         1700     1700           
  Misses        871      871

Impacted Files	Coverage Δ
src/modules/_nomodule/gpuUtils.js	83.87% <100%> (ø)	⬆️

[harshkhandeparkar]

The installation errors should also be fixed now. Awesome!

[robertleeplummerjr]

Woot!

[harshkhandeparkar]

😄

…

On Sat, 21 Dec, 2019, 9:51 PM Robert Plummer, ***@***.***> wrote: Woot! — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1374?email_source=notifications&email_token=AIJI5H7GYD4SSPDVFTQ547LQZY7BNA5CNFSM4J5NAEKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHO7A4Q#issuecomment-568193138>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIJI5H7KGFHOW3KBQ5VGRH3QZY7BNANCNFSM4J5NAEKA> .

[jywarren]

This is awesome!!!!

[keshav234156]

Awesome!!

[keshav234156]

@harshkhandeparkar https://travis-ci.org/publiclab/image-sequencer/jobs/628126608#L1408
Why are we getting Bad image Path

[harshkhandeparkar]

We'll have to fix this separately

…

On Sun, 22 Dec, 2019, 12:10 PM keshav234156, ***@***.***> wrote: @harshkhandeparkar <https://github.com/HarshKhandeparkar> https://travis-ci.org/publiclab/image-sequencer/jobs/628126608#L1408 Why are we getting Bad image Path — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1374?email_source=notifications&email_token=AIJI5H7YBNCRHOLLCJD2WXLQZ4DVBA5CNFSM4J5NAEKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHPJW3Q#issuecomment-568236910>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIJI5HYOBQA7V3WLIDAQY73QZ4DVBANCNFSM4J5NAEKA> .