search

roguedarkjedi / dogcam-ai

The AI based camera focus controller for dogcam
MIT License
1 stars 0 forks source link

Clean up the webcam streamer class #20

Open roguedarkjedi opened 3 years ago

roguedarkjedi commented 3 years ago

It's a bit of a mess and could be done better with some more robust reconnection code and overall better architecture.

Also needs more comments.

roguedarkjedi commented 3 years ago

Building upon this, it would be nice to drop the __LastReadTime and __HasBeenFlushed variables. They really only serve to limit some operations (specifically resize, which may not be as costly [needs some research]).

roguedarkjedi commented 3 years ago

The variables above are really just adding more artificial delay to the overall process which muddles up the entire processing architecture. Adding in the ability to test the unlocked capture rate akin in commit bb2f71b would let us know if we can proceed forward with such a feature.

roguedarkjedi commented 3 years ago

Variables have been dropped as of d02b823b08ec7a5bda338571e7d3641b71f158d7