Skip to content
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

Does this work with 60A Renogy Rover? #12

Open
st4rgut22 opened this issue Feb 11, 2021 · 3 comments
Open

Does this work with 60A Renogy Rover? #12

st4rgut22 opened this issue Feb 11, 2021 · 3 comments
Assignees

Comments

@st4rgut22
Copy link

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 corbinbs self-assigned this Feb 12, 2021
@corbinbs
Copy link
Owner

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
Copy link
Owner

👋 @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
Copy link
Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants