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

strange modbus issue with custom/structure #114282

Closed
commentsonly opened this issue Mar 27, 2024 · 3 comments
Closed

strange modbus issue with custom/structure #114282

commentsonly opened this issue Mar 27, 2024 · 3 comments
Assignees

Comments

@commentsonly
Copy link

The problem

tried to use a a rather longish structure and it fails.
if splitting it up into chunks everything works so I would believe it's a not a problem with types but something to do with the length. Or perhaps a timeout or such like.
Couldn't find any remarks about length limits, any idea?

What version of Home Assistant Core has the issue?

core-2024.3.3

What was the last working version of Home Assistant Core?

none I would remember

What type of installation are you running?

Home Assistant OS

Integration causing the issue

modbus

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

structure: ">2H4h1L1h1H"
structure: ">1H16s3H2I1h1H1I"
structure: ">1H16s3H2I1h1H1I"
structure: ">1H16s3H2I1h1H1I"
structure: ">1H16s3H2I1h1H1I"
works if being separated into 5 sensors

structure: ">2H4h1L1h2H16s3H2I1h1H1I1H16s3H2I1h1H1I1H16s3H2I1h1H1IH16s3H2I1h1H1I"
fails if trying to do it in one sensor

Anything in the logs that might be useful for us?

nothing I could think of

Additional information

No response

@home-assistant
Copy link

Hey there @janiversen, mind taking a look at this issue as it has been labeled with an integration (modbus) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of modbus can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign modbus Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


modbus documentation
modbus source
(message by IssueLinks)

@janiversen
Copy link
Member

janiversen commented Mar 28, 2024

Does not seem to be a bug in the modbus integration. Please remember this channel is for bugs not support.

There is a length limit in HomeAssistant in general, something like 250 characters.

@janiversen
Copy link
Member

If you think there is a bug in the modbus integration, then please follow the guidelines, add a small configuration showing the problem, together with a debug as pr modbus integration documentation.

@github-actions github-actions bot locked and limited conversation to collaborators Apr 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants