Implements continuous mode

[Smankusors]

Fixes #19
Also with an example.

Tested it on Nvidia Jetson Nano with 2 VL53L0X (but it's not from Adafruit).

[kattni]

@Smankusors Please let me know your status on this PR. Let us know if you're stuck or need assistance with anything. Thanks for the contribution!

[Smankusors]

@Smankusors Please let me know your status on this PR. Let us know if you're stuck or need assistance with anything. Thanks for the contribution!

hmm to be honest I'm actually don't want to implement __enter__ and __exit__, kinda confusing. 😵

Should it create new class to handle the context or just use self to handle it? What about multiple sensors? Should it managed by one context manager or managed by itself separately?

thanks for replying btw 😉

[kattni]

@tannewt Can you weigh in on this? It's outside my knowledge. Thanks!

[tannewt]

Should it create new class to handle the context or just use self to handle it? What about multiple sensors? Should it managed by one context manager or managed by itself separately?

I'd just use self. Multiple sensors could either do nested with or do start_continuous and stop_continuous explicitly.

[Smankusors]

I'd just use self. Multiple sensors could either do nested with or do start_continuous and stop_continuous explicitly.

Hmm alright. But unfortunately I left my device and sensor on my workplace. And there's still pandemic going on. I will continue to work on this when I got access back.

[jposada202020]

Hello @Smankusors have you been able to get access to the device, are you still interested in this PR? let us know. Thanks :)

[Smankusors]

Hello @Smankusors have you been able to get access to the device, are you still interested in this PR? let us know. Thanks :)

uh oh, yeah I do already have access to the device, but then I got busy with other things. I will try to find some free time to continue on this. Thanks for the reminder.

[jposada202020]

@Smankusors Thanks :)

[Smankusors]

alright I just implemented the context manager for the continuous mode. Also I add an example for the simple continuous 😉

hmm it seems like the code I wrote kinda look weird, I defined a class (continuous_manager) inside a function... is that ok? Or should I move it to somewhere?

[jposada202020]

@tannewt Could you review please. Thanks

[tannewt]

alright I just implemented the context manager for the continuous mode. Also I add an example for the simple continuous wink

hmm it seems like the code I wrote kinda look weird, I defined a class (continuous_manager) inside a function... is that ok? Or should I move it to somewhere?

I think you can simplify it a bit by having continuous_mode return self and then putting the __enter__ and __exit__ functions on the top level class. __enter__ can call start and __exit__ would call stop.

[Smankusors]

alright I just implemented the context manager for the continuous mode. Also I add an example for the simple continuous wink
hmm it seems like the code I wrote kinda look weird, I defined a class (continuous_manager) inside a function... is that ok? Or should I move it to somewhere?

I think you can simplify it a bit by having continuous_mode return self and then putting the __enter__ and __exit__ functions on the top level class. __enter__ can call start and __exit__ would call stop.

I see... how about the failing checks? Should I add the... license? Copy paste from other files?

[jposada202020]

@Smankusors for the copyright license yes, that is the way to go, as you adding the example you will need to add your name, or if you prefer you github user.
run Pylint and Black locally on your code. You can do
this manually or using pre-commit. Instructions are available here: https://adafru.it/check-your-code

[Smankusors]

OK I simplified the manager, and also add the license to my example codes. Is the license format I entered correct?

[tannewt]

Looks good to me! Thank you!

[FoamyGuy]

tested successfully with feather RP2040 and VL53L0X