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

Shuffleboard Debug Mode vs Non Debug Mode #78

Merged
merged 12 commits into from
Mar 19, 2024
Merged

Conversation

stewluj
Copy link
Contributor

@stewluj stewluj commented Mar 1, 2024

Logs more or less depending whether or not debug mode is true

Moving stuff around lol smile
grub
Copy link

@bobtseattle bobtseattle left a comment

Choose a reason for hiding this comment

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

Noting a mismatch between the label and data.

@kirbt kirbt reopened this Mar 10, 2024
TAKBS2412
TAKBS2412 previously approved these changes Mar 18, 2024
@TAKBS2412
Copy link
Contributor

Approved (pending formatting issues).

@kirbt kirbt dismissed bobtseattle’s stale review March 19, 2024 01:49

everything is already addressed

Copy link

@bobtseattle bobtseattle left a comment

Choose a reason for hiding this comment

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

LGTM

@kirbt kirbt merged commit a24da4b into robototes:main Mar 19, 2024
2 checks passed
Comment on lines +280 to +283
Shuffleboard.getTab("Launcher")
.addDouble("Bottom FlyWheel Temp", () -> launcherBottomMotor.getMotorTemperature());
Shuffleboard.getTab("Launcher")
.addDouble("Top FlyWheel Temp", () -> launcherTopMotor.getMotorTemperature());
Copy link
Contributor

Choose a reason for hiding this comment

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

Have we tested the placement of these? Bottom previously had .withPosition(0, 3) and .withSize(2, 1) and top previously had .withPosition(2, 3) and .withSize(2, 1), but without those I'm worried they will be assigned a spot that overlaps with something else.

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