Skip to content

aalind told me to do this i ha5e this key board#2484

Merged
yuvrajdhadwal merged 1 commit intoRoboJackets:ros2from
CameronLyon:pid_constants
Feb 2, 2026
Merged

aalind told me to do this i ha5e this key board#2484
yuvrajdhadwal merged 1 commit intoRoboJackets:ros2from
CameronLyon:pid_constants

Conversation

@CameronLyon
Copy link
Contributor

Description

Describe your pull request.

Associated / Resolved Issue

Resolves # or ClickUp card

Design Documents

Link

Steps to Test

Test Case 1

  1. Step 1
  2. Step 2
  3. Step 3

Expected result:???

Key Files to Review

Group 1

  • File 1
  • File 2

Group 2

  • File 3
  • File 4

Review Checklist

  • Docstrings: All methods and classes should have the file appropriate docstrings which follow the guidelines in the "Contributing" page of our docs.
  • Remove extra print statements: Any print statements used for debugging should be removed
  • Tag reviewers: Tag some people for review and ping them on Slack

(Optional) Sub-issues (for drafts)

Note: if you find yourself breaking this PR into many smaller features, it may make sense to break up the PR into logical units based on these features.

  • Step 1
  • Step 2

Copy link
Contributor

@yuvrajdhadwal yuvrajdhadwal left a comment

Choose a reason for hiding this comment

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

adsf

DEFINE_FLOAT64(params::kMotionControlParamModule, rotation_ki, 0.0,
"Ki for rotation ((rad/s)/(rad*s))");
DEFINE_FLOAT64(params::kMotionControlParamModule, rotation_kd, 0.0,
DEFINE_FLOAT64(params::kMotionControlParamModule, rotation_kd, 0.9,
Copy link
Contributor

Choose a reason for hiding this comment

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

LGTM

@yuvrajdhadwal yuvrajdhadwal merged commit 5a41f1f into RoboJackets:ros2 Feb 2, 2026
2 checks passed
Squid5678 pushed a commit that referenced this pull request Feb 2, 2026
Co-authored-by: sanatd33 <saisanat@gmail.com>
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.

5 participants