-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
After the update it gives me a result in mm when the result is in cm #6
Comments
@Serrallonga25 |
Maybe the sensor is broken? I use it in a diesel tank to measure the liters I have left
If it measures correctly for you, the sensor may be failing, as I have noticed that it gets stuck after a while and says it has failed. But it seems strange to me that it truncates a digit in the result. |
few questions What is purpose of So sensor is giving 97mm and you are multiplying by 10 to make it 970mm, so I am guessing you were expecting 970mm is that right. I check the sensor by aiming it at a flat surface at a few different distances. Is your range stays always 0 if not that is a problem with the conditions the sensor is measuring in I am interested that it get stuck and fails - can you give me any logs of a fail. There a quite a few error msg depending on cause of fail |
Also are you using IDF or arduino and what esp are you using |
I have downloaded the old version from 2 months ago and I also get the truncated result but it gives me a different value
|
|
Framework type - but by logs it is arduino. The difference between 97 and 80 with old code is not the code, it is accuracy of sensor, so that's ok. So your sensor is reading 79mm (or 80mm) |
The correct measurement is 780mm, not 78mm
I don't know what happened, but in some update eshome broke or maybe the sensor broke. I'll try another board that I have to see if it's the esp32cam that has always been a bit of a fool. |
Now it measures correctly I think I wasted your time, sorry, all I did was clean the sensor casing. But since it coincided with the update, I got confused
Output without filter:
Let's see if the blocks disappear too. |
Good morning, now it takes the correct measurement but it continues to block
|
The issue seems to be that there is an error reading a register ( for data ready) which creates a warning but then the it fails. You were getting a similar issue before you cleaned the sensor. I have not seem that issue on my sensor, let me have a look at the code and see if I can make a change that might help not cause the sensor to fail. The blocking issue is not really an issue, but a symptom of reads or writes taking longer than they should. Busy tonight but I will get back to you in the next day or so and let you know what I have changed. Thanks for all the info on how your sensor is performing with my component |
@Serrallonga25 |
Thanks @mrtoy-me , I'll try it and let you know. |
Hi @mrtoy-me , I only get the following warning from time to time (but it happens to me on other sensors)
Thanks |
It seems like he was waiting for me to say there was no problem.
|
Thanks so much for doing the testing and providing the error codes. Was the component failing after a day or two prior to the recent update removing the vl53l0x hub setup ? |
Maybe it had failed before but I hadn't detected it or because I wasn't looking at it so closely I hadn't noticed. But what surprises me is that you tell me that it doesn't fail for you. It could be the fault of my sensor from AliExpress and it doesn't have enough quality. At the moment it is working again after restarting. Thanks |
I have got it to fail not sure if it is same cause as you yet - so I am doing more testing and will make some changes when finished - will let you know |
Thanks Today I found the error again and had to restart.
|
Have not forgotten - I have been making incremental changes and testing along the way but it takes a few days to test each time to see if it fails - I am not on my usual test system and I am getting some fails after a while so still working on it. Will let you know when I have something. |
Thank you for your effort |
@Serrallonga25
So try above with the updated component and let me know how it goes. I will keep testing and cleanup the code when I am have done some more testing. |
Hi, when downloading the latest version it gets stuck on startup and does not initialize any type of connection, I can't even see the logs. I'll try to compile it on another board that I have to see if I can see the exact error via USB. I have now configured the frequency and timeout with the latest version that worked, let's see how it goes. I'll keep you informed and thanks. |
Apologies, I have been offline - my sensor failed after just over 6 days on 200khz - I am trying 400khz now. So you might want your hold off testing for a while until I do more. Starting to think it is a hardware issue because the i2c seems to stop working but I will persist. I am thinking of converting the polulo library to esphome and see how that goes. |
Hi, I have reinstalled the previous version from February before updating and I have had two days without errors, now I have compiled it by mistake and I have restarted the device but it seems to be the most stable one, I do not get errors or crashes. This is the configuration.
If you need me to do any testing, let me know. |
Thankyou - just so I confirm that I understand, you are running a version from February and it is ok so far? Thanks for the offer to test, I will get you a version which uses February code but I will leave it a few weeks probably. Let me know if what you are you are running now has any issues- Thanks. |
What version of esphome are you using to compile? |
Today I got the latest version, so I compiled it without realizing it. It had been working fine for 2 or 3 days.
202402 esphome-my-components-54ed55e0349bec50aa7321a0243073705745ec24.zip I get some warnings but it seems to work fine, I am attaching the version I compiled now. |
@Serrallonga25 |
Yes, the vl53l1x is correct. |
Hi, @mrtoy-me |
Thank you for the feedback, appreciate it. I have one running about 4.5 days now with same underlying i2c read/write as you are running but idf with 400mhz i2c. |
Hi @mrtoy-me,
But after the update vl53l1x I get an incorrect result.
It gives me 67,000mm instead of 670mm
Could it be my fault or some bad configuration?
Thanks for your work.
The text was updated successfully, but these errors were encountered: