search

corbinbs / solarshed

A Python library to help monitor solar charge controllers typically used in off the grid applications.
MIT License
86 stars 24 forks source link

Does this work with 60A Renogy Rover? #12

Open st4rgut22 opened 3 years ago

st4rgut22 commented 3 years ago

Hi, thanks for this project. I have 60A MPPT Renogy Rover and am trying to communicate with a MacOS via RS232 cable <-> DB9 connector. When I run renogy_rover.py I get this error: 

"/Users/ed/Desktop/solarshed-master/solarshed/controllers/modbus/minimalmodbus.py", line 1699, in _extract_payload
    MINIMAL_RESPONSE_LENGTH_RTU, response
modbus.minimalmodbus.InvalidResponseError: Too short Modbus RTU response (minimum length 4 bytes). Response: 'ÿ'

I double checked my wiring, and am not sure what to do next. Does 60A Mppt controller work on a different protocol than 20A/40A version?

corbinbs commented 3 years ago

Hi @st4rgut22 !

I currently only have access to a 40A Renogy Rover. I used it to help develop this library. I would think that all of the Rover editions would communicate using the same protocol. 🤔

I'm behind on issues/PR reviews on the project. I just noticed that a couple of PRs have been opened up about changes that have been made in new(er) modbus versions that could impact the library. (See #9 and #10)

I'll see if I can help 👀 into this more over the weekend. I may be able to suggest a few things you might try

Thanks,

Brian

corbinbs commented 3 years ago

👋 @st4rgut22 - I've gotten those open PRs merged along with some updates to the project dependencies. I'm behind on testing those out against my own set up before releasing a new library version.

I'm 🤞 that I can get some runtime on the new version of the library ASAP and will circle back to this issue once the new library version is released to see if it fixes up this error. Thanks for your patience! 🙇

st4rgut22 commented 3 years ago

I tested on 40A Rover too and have the same issue. I used several kinds of USB - Serial converters. Perhaps Renogy has found a way to prevent this communication because the Renogy BT1 module communicates fine.