-
-
Notifications
You must be signed in to change notification settings - Fork 34
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
Request and decode warning/alarms (0x44
) frame
#6
Comments
0x44
) frame
0x44
) frame0x44
) frame
|
Hi! I managed to decode the alarms, but I can't send two messages to modbus. I've tried to send 2 commands in the update method in seplos_bms.cpp but this seems to break down the first command as i get no data in the response. Could you advice how to process two commands? I guess the code need to be changed to send one command, wait for response, process it and then send a second command and process it. |
Could you share some more details? In best case some could. So it's easier for me to provide a good solution. |
Yes, what i'm trying to do is to send 0x42 packet and then 0x44. In the file seplos_bms.cpp i've tried the following:
After that, i'm getting the following response from the bms: I think that the update method should be something like:
|
Please add a
All responses will be reported as the mentioned warning because there is no decoder available at the moment for this kind of response.
|
The delay option is working! Right now the differentiation between telemetry and non telemetry frames is done by this condition
Which is basically looking at size of the response and the number of cells, but the telecommand frames also have this data, so the |
Yes. The implementation must be improved on the long run here. We have to remember the last transmitted request and expect the dedicated response to avoid guessing the frame type by length. |
Hello, i'm not a coder but found something maybe could help (please delete my post if useless) Have a good day ! :) |
Hello, i supposed you point me to the part of code which is interesting but sorry i'm not a coder and i don't know what to do with this ..... |
That's fine. The link to the explicit method is just a pointer for myself. |
The text was updated successfully, but these errors were encountered: