aiortc icon indicating copy to clipboard operation
aiortc copied to clipboard

WIP: Computer vison streaming demo

Open kodonnell opened this issue 5 years ago • 4 comments

I won't write too much here as there's a bunch in the readme.md and you should be able run the demo code easily to see what it's talking about. But I think it's pretty cool (naturally) and will be useful to a bunch of people.

Status:

  • Code is working aside from some heisenbugs (as documented in the readme). I wouldn't mind someone who knows about webrtc/aiortc to run their eyes over it to see if I'm doing it the "right" way, or if there are more optimal ways. Possibly @jlaine (partly as I'm also hoping it'll tickle your fancy).
  • Docs etc. need to be updated with some screenshots etc. I've written them from my perspective (e.g. "I'm a novice and don't trust me") which we might want updated if it's going in here (e.g. "less novice people have reviewed this and OK'd it").

kodonnell avatar Jul 14 '20 12:07 kodonnell

Codecov Report

Merging #392 into main will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #392   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           31        31           
  Lines         5492      5492           
=========================================
  Hits          5492      5492           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update c0504b6...8daa20e. Read the comment docs.

codecov[bot] avatar Jul 14 '20 12:07 codecov[bot]

Do you plan to complete this PR or should I close it for now?

jlaine avatar Mar 02 '21 15:03 jlaine

Hmmm I think it was actually finished IIRC. I'm probably not going to put any more time into it - up to you whether it's useful as-is for an example (even if not polished).

kodonnell avatar Mar 02 '21 21:03 kodonnell

@jlaine can we close this PR?

rprata avatar Dec 01 '23 12:12 rprata