tomyyyD
commented
1 year ago all requested changes and sensor is set to the 100ms setting, except I did not change lux to be async, I left a comment above as to why, but in summary I believe that because lux is returning a value that we rely upon we will need to wait at some point in our code for it. We cannot do estimation without the lux, so no matter if the function is async we need to wait for a value.
Adding the drivers for the sun sensor and a system check test for the new sun sensors. The system check test is exactly the same as the pervious sun sensors except lux has to be called as a function for the new sensors. Sun vector seems to be returning the correct values in my testing with a phone flashlight, but the numbers seemed a bit inconsistent and jumped around a lot. Would be appreciated if someone could take a look and see if their might be any problems.