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

Increase broadcast and unicast table sizes #71

Merged
merged 2 commits into from
Jun 21, 2024

Conversation

puddly
Copy link
Collaborator

@puddly puddly commented Jun 6, 2024

  • The broadcast table size increase should help with NETWORK_BUSY errors. This is the table EmberZNet uses to determine if there are too many broadcasts happening at once.
  • The unicast table should help enqueue more packets.

@puddly
Copy link
Collaborator Author

puddly commented Jun 6, 2024

For ease of use, here are firmwares for some common sticks:

GitHub does not allow .gbl files so I have zipped them. Please unzip them and flash them with an appropriate tool:

@puddly
Copy link
Collaborator Author

puddly commented Jun 6, 2024

@jclendineng Let's move firmware discussions over here. If your firmware is showing up as 7.4.0.0, it didn't flash. It'll show up as Firmware: 7.4.2.0 build 0.  

@puddly
Copy link
Collaborator Author

puddly commented Jun 21, 2024

Feedback has been positive so far. Merging.

@puddly puddly merged commit 452165e into NabuCasa:main Jun 21, 2024
16 checks passed
@evelant
Copy link

evelant commented Jul 4, 2024

Per home-assistant/core#86411 this is a massive improvement but seems to have introduced an issue with group behavior. To summarize from that thread:

Random members of groups don't respond to group commands. Which members of the group fail to respond is random with each command sent. This never happened prior to this build. Groups addressed via a binding on an inovelli blue switch always respond in unison. Only commands to groups from the coordinator since this build exhibit the random failure. I'm not sure how the changes made in this build could have introduced this but I am certain this issue was introduced in this build.

@jclendineng
Copy link

jclendineng commented Jul 4, 2024 via email

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

Successfully merging this pull request may close these issues.

4 participants