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

Large number of dropped MQTT messages during device list scan. #435

Open
medrapal opened this issue Oct 18, 2024 · 7 comments
Open

Large number of dropped MQTT messages during device list scan. #435

medrapal opened this issue Oct 18, 2024 · 7 comments

Comments

@medrapal
Copy link

Problem/Motivation

(Why the issue was filed)

EQMX shows a large number of dropped MQTT messages during a device list scan. I have about 70 devices and EQMX reports about 525 dropped MQTT messages during the scan. Device list scan works and I see current status for all devices in the list.

Expected behavior

Maybe zero? Or at least very few.

(What you expected to happen)

Actual behavior

(What actually happened)

Steps to reproduce

(How can someone else make/see it happen)

Proposed changes

To be fair, this is a low-priority issue as you can't seen any issues unless you happen to have EQMX installed (thank you for the great HA addon).

(If you have a proposed change, workaround or fix,
describe the rationale behind it)

Copy link

There hasn't been any activity on this issue recently, so we clean up some of the older and inactive issues.
Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thanks!

@github-actions github-actions bot added the stale There has not been activity on this issue or PR for quite some time. label Nov 18, 2024
@inverse
Copy link
Contributor

inverse commented Nov 18, 2024

@medrapal If you upgrade to the latest version does your issue persist? I've added a pool size to the requests that the service issues so that it shouldn't spam the network with more than 50 requests by default.

This option is configurable in the config page too, as well as other options such as connect and read timeouts.

@medrapal
Copy link
Author

medrapal commented Nov 18, 2024 via email

@inverse
Copy link
Contributor

inverse commented Nov 18, 2024

Not an expert but do other activities on the network cause such packet loss? Or is it purely this scan?

@medrapal
Copy link
Author

medrapal commented Nov 18, 2024 via email

@inverse
Copy link
Contributor

inverse commented Nov 18, 2024

What happens when you set Request concurrency to something low?

@inverse
Copy link
Contributor

inverse commented Nov 18, 2024

Ahh I think I know what it could be. The JS on the frontend still issues a request for each device, and doesnt respect the concurrency that was applied to the backend. Check the network tab of your browser on that page.

Ill see what I can do, meanwhile try changing the Refreshtime to something higher.

@github-actions github-actions bot removed the stale There has not been activity on this issue or PR for quite some time. label Nov 19, 2024
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