machinebox / webFaceID

Use HTML5, Go and Facebox to implement face verification on a website
Apache License 2.0
59 stars 14 forks source link

cant run demo. #1

Open najamsk opened 5 years ago

najamsk commented 5 years ago

what command is required with command line arguments to run this demo? go run main.go throws following error.

# command-line-arguments
./main.go:32:9: undefined: NewServer
matryer commented 5 years ago

My guess would be that you're using go run to run it. If you do go run main.go it will only load main.go and ignore anything in other files.

To run it properly, try building it first (which will include all files) with: go build -o demo and then run the newly created demo binary: ./demo.

Hope this helps.

matryer commented 5 years ago

or you can try go run *.go which will also select all .go files.

najamsk commented 5 years ago

SyntaxError: missing ) after argument list[Learn More] semantic.min.js:16:3023 console showing this.

najamsk commented 5 years ago

it asks for go cam permission that i give. but nothing shows up in video player so i checked console which was showing following error.

SyntaxError: missing ) after argument list[Learn More]

semantic.min.js:16:3023

najamsk commented 5 years ago

replacing following video.src = window.URL.createObjectURL(stream); with video.srcObject = stream; seems to work in firefox.

matryer commented 5 years ago

Strange, so I guess that API isn’t stable yet?!

On Mon, 25 Feb 2019 at 12:09, najamsk notifications@github.com wrote:

replacing following video.src = window.URL.createObjectURL(stream); with video.srcObject = stream; seems to work in firefox.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/machinebox/webFaceID/issues/1#issuecomment-466988375, or mute the thread https://github.com/notifications/unsubscribe-auth/AAGNG-6SHG3IbCRoTAkJtLPfT1EO-Hi0ks5vQ9JrgaJpZM4bPJLa .

najamsk commented 5 years ago

nope. i tried to ask co-worker to show their face on cam. since didnt train model on their face. i was expecting some error message but instead the camera icon truns into loading and keep doing loading stuff. i check console it was saying button.transition is undefined. so to solve that i change more code by showing text message. maybe run demos again in cross browsers so people can actually see them working and hop into stuff. Having said that i still enjoy browsing through demos and articles on medium.

matryer commented 5 years ago

Yeah it’s annoying that it doesn’t work in your browser. I’m sorry about that.

They were all tested in Chrome, and are just examples so are unlikely to get cross-browser QA testing etc like we do with our production services.

Good job for making it work though - skillz.

Mat

On Mon, 25 Feb 2019 at 12:23, najamsk notifications@github.com wrote:

nope. i tried to ask co-worker to show their face on cam. since didnt train model on their face. i was expecting some error message but instead the camera icon truns into loading and keep doing loading stuff. i check console it was saying button.transition is undefined. so to solve that i change more code by showing text message. maybe run demos again in cross browsers so people can actually see them working and hop into stuff. Having said that i still enjoy browsing through demos and articles on medium.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/machinebox/webFaceID/issues/1#issuecomment-466992194, or mute the thread https://github.com/notifications/unsubscribe-auth/AAGNG1w_srZNdzUMMPfYoRtwUdsuKPwQks5vQ9XLgaJpZM4bPJLa .

najamsk commented 5 years ago

i check on my co-workers machines using chrome still same errors. I think underlaying apis for html5 stuff got changed and they require minor fixes to work as expected.

matryer commented 5 years ago

Hmm, in that case, do you fancy submitting a PR with your fixes? We’d love to improve the experience for others if we can.

On Mon, 25 Feb 2019 at 12:27, najamsk notifications@github.com wrote:

i check on my co-workers machines using chrome still same errors. I think underlaying apis for html5 stuff got changed and they require minor fixes to work as expected.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/machinebox/webFaceID/issues/1#issuecomment-466993078, or mute the thread https://github.com/notifications/unsubscribe-auth/AAGNG6-ozDntZ5J_AbWWVK-oIsIRMsZWks5vQ9angaJpZM4bPJLa .

najamsk commented 5 years ago

my pull request will be broken as its not working in chrome. but if i make it work in chrome sure.

najamsk commented 5 years ago

one more thing. is there any forum or channel where devs can ask your team questions or report issues. asking on twitter is not that great and engaging founder for small code stuff is like wasting your time. thanks for your time, responses and your efforts.

aaronedell commented 5 years ago

Hi @najamsk - can you join the Machine Box slack here: https://machineboxslack.herokuapp.com/

najamsk commented 5 years ago

Hi @najamsk - can you join the Machine Box slack here: https://machineboxslack.herokuapp.com/

joined the community slack, thanks for sharing. let me create a pull request.

najamsk commented 5 years ago

Hmm, in that case, do you fancy submitting a PR with your fixes? We’d love to improve the experience for others if we can. On Mon, 25 Feb 2019 at 12:27, najamsk @.***> wrote: i check on my co-workers machines using chrome still same errors. I think underlaying apis for html5 stuff got changed and they require minor fixes to work as expected. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AAGNG6-ozDntZ5J_AbWWVK-oIsIRMsZWks5vQ9angaJpZM4bPJLa .

pull request created. I have never contributed in open source stuff and don't know about creating pull request. by clicking few buttons i manage to create pull request that I hope somewhat close to your expectations.

najamsk commented 5 years ago

you can find my pull request at https://github.com/machinebox/webFaceID/pull/2