Skip to content

Conversation

@hnakata-JP
Copy link
Contributor

@hnakata-JP hnakata-JP commented May 29, 2024

Description

In this pull request, I changed the max current setting about the wire grid kikusui agent. It was changed from 3.0 A to 4.5 A.
The kikusui power supply itself supports 18 V-5 A max in its specification. And this change will increase the motor's max torque, so the wire grid can rotate under lower temperatures.

Motivation and Context

Now the wire grid used for SAT-MF2 has an issue in that it cannot rotate under low ambient temperature, even though the grease in its bearing has temperature resistance. When I checked the rotation directly, the rotator can move with a small additional torque support by hand. Then, to make the rotation robust, we want to change the max current setting to 4.5 A.

How Has This Been Tested?

At the site, I changed the settings of the kikusui power supply manually, and confirmed the grid can rotate at 4.5 A max. Further, the over current protection in the hardware is now set at 4.9 A. So this change makes the rotation easy for the OCS operation as well.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

Copy link
Member

@BrianJKoopman BrianJKoopman left a comment

Choose a reason for hiding this comment

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

This looks good to me, I'm assuming since you've tested this that 12V is high enough to drive the new higher current.

@BrianJKoopman BrianJKoopman merged commit dcdcd21 into main May 30, 2024
@BrianJKoopman BrianJKoopman deleted the wiregrid_kikusui_debug branch May 30, 2024 13:25
@hnakata-JP
Copy link
Contributor Author

12 V is the value determined by the company. It should be enough for the motor.

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