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

fix(hardware): remove can messenger motor_enabled listener #14943

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

ahiuchingau
Copy link
Contributor

Overview

When #14479 was added, the motor enable message listener was never cleaned up properly and thus causing the robot to slow down significantly as it runs more and more move commands.

@ahiuchingau ahiuchingau marked this pull request as ready for review April 17, 2024 22:46
@ahiuchingau ahiuchingau requested a review from a team as a code owner April 17, 2024 22:46
Copy link
Contributor

@vegano1 vegano1 left a comment

Choose a reason for hiding this comment

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

Nice,
The listener count was at 64 and incrementing before this change, now it's about 2-3 and not incrementing. The bot is also not super slow anymore.

Copy link
Member

@sfoster1 sfoster1 left a comment

Choose a reason for hiding this comment

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

ah, nice

@ahiuchingau ahiuchingau merged commit 7d6100d into edge Apr 18, 2024
19 checks passed
@ahiuchingau ahiuchingau deleted the hardware_remove-listener branch April 18, 2024 15:37
Carlos-fernandez pushed a commit that referenced this pull request May 20, 2024
<!--
Thanks for taking the time to open a pull request! Please make sure
you've read the "Opening Pull Requests" section of our Contributing
Guide:


https://github.com/Opentrons/opentrons/blob/edge/CONTRIBUTING.md#opening-pull-requests

To ensure your code is reviewed quickly and thoroughly, please fill out
the sections below to the best of your ability!
-->

# Overview
When #14479 was added, the motor enable message listener was never
cleaned up properly and thus causing the robot to slow down
significantly as it runs more and more move commands.
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.

3 participants