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 2160 motor driver register usage #755

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

pmoegenburg
Copy link
Contributor

This corrects the 2160 motor driver register we use with the update_glob_scaler method.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (65d6a31) 83.58% compared to head (90b080b) 83.58%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #755   +/-   ##
=======================================
  Coverage   83.58%   83.58%           
=======================================
  Files         101      101           
  Lines        4044     4044           
=======================================
  Hits         3380     3380           
  Misses        664      664           

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.

Looks good, but what effects does this have? This would alter reading the current of the tmc2160, but it wouldn't alter what current is actually used, right?

Copy link
Contributor

@ahiuchingau ahiuchingau left a comment

Choose a reason for hiding this comment

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

Nice catch. I don't think it's being used anywhere, could you verify that @pmoegenburg?

@pmoegenburg
Copy link
Contributor Author

Confirmed this method isn't being used anywhere.

@pmoegenburg pmoegenburg merged commit 128b936 into main Feb 21, 2024
35 checks passed
@pmoegenburg pmoegenburg deleted the fix_motor_driver_register_naming branch March 13, 2024 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants