Skip to content

[WIP] Add console warnings for high MQTT publish rates - #238

Closed
mwfarb with Claude wants to merge 1 commit into
masterfrom
claude/add-console-warnings-for-high-mqtt-rates
Closed

[WIP] Add console warnings for high MQTT publish rates#238
mwfarb with Claude wants to merge 1 commit into
masterfrom
claude/add-console-warnings-for-high-mqtt-rates

Conversation

@Claude

@Claude Claude AI commented Mar 12, 2026

Copy link
Copy Markdown

Thanks for assigning this issue to me. I'm starting to work on it and will keep this PR's description up to date as I form a plan and make progress.

Original prompt

This section details on the original issue you should resolve

<issue_title>high mqtt publish rates should log console warnings</issue_title>
<issue_description>The overall goals is to discourage users from overloading the Mqtt network with unnecessary messages. We'd like to cap those messages ideally before publishing. Any message we deny publishing for performance purposes we should warn the user of the reason. We also want to do be wary of the overhead cost of evaluating each message which may cost local performance. Can we:

  • globally deny messages over 100ms average over 1 second per object id, or is it better to only warn if the threshold is crossed?
  • globally deny messages with the same json payload, sans timestamp, since they are not providing any updates to the scene graph?</issue_description>

Comments on the Issue (you are @claude[agent] in this section)

@Claude Claude AI assigned Claude and mwfarb Mar 12, 2026
@Claude
Claude AI requested a review from mwfarb March 12, 2026 21:52
Copilot stopped work on behalf of mwfarb due to an error March 12, 2026 21:52
@mwfarb

mwfarb commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

@claude[agent] did you have any proposal?

@mwfarb mwfarb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@claude[agent] This PR is empty. Please begin again.

@Claude
Claude AI requested a review from mwfarb March 13, 2026 15:13
Copilot stopped work on behalf of mwfarb due to an error March 13, 2026 15:13
@mwfarb mwfarb closed this May 14, 2026
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.

high mqtt publish rates should log console warnings

2 participants