Skip to content

fix(subsystems): add missing Logger.processInputs() calls in periodic() #15

Description

@coderabbitai

Problem

Several subsystems call io.updateInputs(ioInputs) inside periodic() but never forward the refreshed inputs to AdvantageKit's logging pipeline via Logger.processInputs(). This means the input values are silently dropped and never logged.

The correct pattern (already used in SwerveDriveSubsystem) is:

override fun periodic() {
    io.updateInputs(ioInputs)
    Logger.processInputs("<SubsystemName>", ioInputs)
}

Affected Subsystems

File Missing Call
src/main/kotlin/org/frc5183/robot/subsystems/collector/CollectorSubsystem.kt Logger.processInputs("Collector", ioInputs)
src/main/kotlin/org/frc5183/robot/subsystems/shooter/ShooterSubsystem.kt Logger.processInputs("Shooter", ioInputs)
src/main/kotlin/org/frc5183/robot/subsystems/vision/VisionSubsystem.kt Logger.processInputs("Vision", inputs)
src/main/kotlin/org/frc5183/robot/subsystems/turntable/TurntableSubsystem.kt Logger.processInputs("Turntable", ioInputs)

Reference

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions