Skip to content

Cleanup+improvements#60

Merged
NinjaBuilderPro merged 10 commits intomainfrom
cleanup+improvements
Feb 9, 2026
Merged

Cleanup+improvements#60
NinjaBuilderPro merged 10 commits intomainfrom
cleanup+improvements

Conversation

@jnallard
Copy link
Copy Markdown
Contributor

@jnallard jnallard commented Feb 9, 2026

Changes

A lot of minor changes in here:

Deprecated logic related to BaseSwerveDrive and PIDTuners

Since we are using AdvantageKit's TalonFx Template, we don't need to maintain our old swerve drive code (hopefully) and we have no need for PIDTuner anymore

DashboardNumber & Unit

  • Updated constructors for DashboardNumber to be more friendly
  • DashboardNumber now uses AdvantageKit's LoggedNetworkNumber so values will be around for replay
  • All values changed now are displayed in a "Changed DashboardNumbers" alert type, which is viewable in Dashboards and AdvantageScope
  • CheckAll no longer needs to be called manually in the 2026 project - now handled by a PeriodTasks subsystem
  • DashboardUnit now uses DashboardNumber to get a lot of features associated with it (like callbacks and alerts)

CTRE Helpers

  • ChaosTalonFxs sims now updated automatically via PeriodicTasks (i.e., simulationPeriodic doesn't need to be called manually in the 2026 subsystems)
  • Tuners now have more default tunables that can be added via builder methods

Screenshot showing off DashboardNumber Alerts and more tunables:

image

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 9, 2026

The 2026.1.1.beta-21839750253 package for this MR can be found here: https://github.com/Manchester-Central/CHAOS-Shared-Code/packages/2033853?version=2026.1.1.beta-21839750253

Copy link
Copy Markdown
Collaborator

@NinjaBuilderPro NinjaBuilderPro left a comment

Choose a reason for hiding this comment

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

LGTM

@NinjaBuilderPro NinjaBuilderPro merged commit 4078c12 into main Feb 9, 2026
1 check passed
@NinjaBuilderPro NinjaBuilderPro deleted the cleanup+improvements branch February 9, 2026 23:20
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.

2 participants