-
Notifications
You must be signed in to change notification settings - Fork 119
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
Vattenfall S34U18 (Sanxing SX631) change #26
Comments
Cool, I will add it to the main branch and update the README with info about the Sanxing SX631 meter 👍 |
Yes, please do that. I have been running mine fine since a week ago now with no issues whatsoever. |
Fixed in #29 👍 |
@Spitfire69 Hi! I have a few questions for you :-) First of all, did you have any problems with getting this to work except from the buffer? I cant get it to work, i get the data from the meter but i get crc fail. What hardware are you using? And iam interrested in what data you get from the meter as i dont get very much from my sanxing (dont have it in front of me right now) but i only think its like 8 lines and no voltages och individual phase data at all. Looking forward for you answer :-) |
Hi, I’m using a Wemos D1 mini and it has been working flawlessly for over a month now. As mentioned the only thing needed was to extend the buffer. However before I did that I got CRC errors so my suggestion is to just add more bytes and test. Good luck! |
Ok, iam using D1 mini, tried increasing buffer to 100 (was 60, but i dont know what it was when you made yours) and still no luck with crc :-( So for now i bypassed the crc check and now i get data to home assistant but the energy dashboard is still blank.. well well..
|
@Spitfire69 [21:43:03][D][data:265]: /AUX5 |
Edit: The output below is of course the output to HA not the output from the meter…I will see if I can fetch that to. If I recall right it was the 0-0:1.0.0 or 0-0:96.1.1 string that was too long (a long ID string) and you do not seem to have the same output at all. Yours is not even close…has Vattenfall installed your or is it a standard Sanxing? If that is the case I guess they have different firmware. Here’s my output: [21:46:43][I][crc:276]: Telegram read. CRC: 84C3 = 84C3. PASS = YES |
@Spitfire69 Can you please paste the meter output instead of what esphome p1-reader sends to home assistant? :-) |
@Slamkryp @Spitfire69 are you using 115200 baud and 8N1 (inverted) ? For some reason, my ESP32C3 cannot read any data from S34U18, but other meters such as L+G E360 work just fine. |
For the S23U18 model (Sanxing SX631) that Vattenfall is distributing now you have to increase the buffer slightly.
I tried with #define BUF_SIZE 60 and it seems sufficent.
The text was updated successfully, but these errors were encountered: