-
Notifications
You must be signed in to change notification settings - Fork 209
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
Puppeteer error: UnhandledPromiseRejectionWarning: Error: Failed to launch chrome! #1032
Comments
Can't we use headless firefox? Is it available? Firefox is a lot more stable as compared to chromium. @VibhorCodecianGupta? |
Puppeteer is chromium specific, it's a subproject of the Chromium project. Switching to Firefox would essentially mean switching to some other web driver like selenium. Let's try resolving this first, if it's too much of an issue, then we can see what comes next. cc @jywarren |
Why don't we switch to firefox? It will fix the problems and also make it future proof. What do you say? |
TL;DR @jywarren from the conversation on the above-mentioned forums, it looks like it was a GCF bug which was solved. People claimed that earlier versions were working for them, like For some reason though, we are still facing this issue. People also mentioned how sometimes redeploying made it work, rebuilding the project and redeploying also made it work. In short, it's flaky. Maybe you could try out a few alternatives given in the issue comment. We're using Node 8 runtime, right? |
We actually have support for nide 6, 8, 10. I had suggested to drop node 6 support and jywarren had agreed. Should we do that? |
@harshkhandeparkar I also think due to some dependencies and other features we may add in future do not work on node v6, it can be dropped! |
great! |
Trying 2 things:
|
But we are still seeing the settings parsing error, so i removed the options string from webgl, and let it run on defaults... |
On the edge one pointed at #1031, i still get:
|
Ooh, actually that's different: |
Erg, meant the reverse. I'll try with Update: OoOOOOooooOO! This is looking promising!
I think we need to build this and merge it, and then puppeteer will use the dist file!!! |
trying this: |
I think this is, technically, running. but it's timing out:
|
And with two images, i'm seeing
A better test input: |
Strange, because when i copy paste it into here, it's valid JSON: https://duckduckgo.com/?q=json+validator&atb=v121-6&ia=answer |
And: > JSON.parse('[{"id":1,"input":"https://i.publiclab.org/i/31778.png","steps":"webgl-distort"},{"id":2,"input":"https://i.publiclab.org/i/31778.png","steps":"webgl-distort"}]')
[ { id: 1,
input: 'https://i.publiclab.org/i/31778.png',
steps: 'webgl-distort' },
{ id: 2,
input: 'https://i.publiclab.org/i/31778.png',
steps: 'webgl-distort' } ] |
Oh, hm, i'm running it with this now and it doesn't seem to be having the same JSON parse error: |
It's not completing... but no errors yet... ahhh, ok, timeout: anyways, next step would be to try to pull in the output of one sequencer into the other, with a sequence like: Actually, with the function code @tech4GT has written, we should be able to point at the output of the earlier step with a number as input, and then use |
Closing this puppeteer issue and moving back to publiclab/image-sequencer-app#6 ! |
The VERY exciting work in #1021 and #1007 has enabled node-based WebGL processing. However, I'm working to get it running on Google Cloud and am encountering this error there, and also running it via the
index.js
CLI, for exampe:./index.js -s webgl-distort -i examples/images/test.png
yields:
I'm also trying to get it to run headless in #1031 because of an issue I found there. Any help appreciated! This is very very close based on great work from several of you! @tech4GT @VibhorCodecianGupta @Divy123 @harshkhandeparkar --
Thanks!
The text was updated successfully, but these errors were encountered: