guardianproject / haven

Haven is for people who need a way to protect their personal spaces and possessions without compromising their own privacy, through an Android app and on-device sensors
https://guardianproject.github.io/haven/
GNU General Public License v3.0
6.58k stars 728 forks source link

Bug: using the slider to configure camera sensitivity, audio floor, vibration baseline #336

Open deviantollam opened 5 years ago

deviantollam commented 5 years ago

on all devices I've tested, whenever I'm configuring the "slide to set the detection threshold" the experience of pressing and dragging on the slider is not clean and smooth...

let's say the value currently shows "50" (the default for sound detection as you start configuring) and you press a finger anywhere on the slider and start to drag. that value will almost without fail jump wildly to some other number, then start tracking up or down as you drag. while this may not be a major pain for microphone or movement sensor config (however, it is still a bit of an unclean UX) it's a bigger issue when trying to configure camera sensitivity.

with such a HUGE range of possible values (does it go from zero to about 30,000?) it's a lot harder to smoothly dial-in to a value you really want here. you can't pick up and re-drag your finger easily on the slider bar because any subsequent tap will re-snap the number displayed to some wildly altered value.

this could be said to be related to comments in #303 and #275 which touch on camera sensitivity settings, but i'm opening this as a bug rather than an enhancement due to the fact that (a) it affects all three threshold settings, not just camera and (b) it's inconsistent with how most UX would be desired, i think

nwainwri commented 5 years ago

Was also looking to see if anyone had brought this up; it's rather confusing as to what the camera sensitivity numbers translate into?

deviantollam commented 5 years ago

I also posted this in #347 but perhaps it applies more to this thread... the Camera Sensitivity settings are really hard to get right in 0.2.0-alpha4, it seems. i filmed a device where i was trying to get the settings tuned adequately...

https://www.youtube.com/watch?v=Sx1oCZ3FCg4

...in the first segment of the video, the Camera Sensitivity value is set around "8500" (whatever that value means) and you can see Haven sees "movement" all over the place (is this caused by things like LED bulbs that have a pulse rate wavelength which is faster than the eye can see but which the phone camera sensor is capturing?)

Then, at the 8 second mark you see the preview after the sensitivity has been adjusted to about "11000" and that "noise" isn't appearing. We think that things might be alright now, yes?

While the device can detect my finger appearing in the field of view... from this perspective it will not detect anyone opening the door and entering the room. I have to cross the room and make it all the way up to about 2' from the phone before it decides to record.

So frustrating. :-(

deviantollam commented 5 years ago

ok... so maybe this isn't related to camera sensitivity... this may be related to when the screen is off. although i have successfully gotten Haven to detect camera events with the screen off (and capture video, send video to me, etc) in the past... something has happened now and it won't work anymore.

i'm going to uninstall 0.2.0-beta4, remove all settings, and try re-installing again. i'm going to troubleshoot the hell out of this today in an attempt to isolate what is preventing the app from doing any real monitoring and detection.

deviantollam commented 5 years ago

i'm doing a lot of testing and putting all my notes into #347