Skip to content

logging: persist AdvantageKit stream to disk with WPILOGWriter - #7

Open
AdamEXu wants to merge 1 commit into
mainfrom
add-wpilog-writer
Open

logging: persist AdvantageKit stream to disk with WPILOGWriter#7
AdamEXu wants to merge 1 commit into
mainfrom
add-wpilog-writer

Conversation

@AdamEXu

@AdamEXu AdamEXu commented Jul 16, 2026

Copy link
Copy Markdown
Member

What

Adds WPILOGWriter as a second AdvantageKit data receiver in Robot.java (2 lines + comment). Real-robot runs now write the full AdvantageKit stream to /U/logs on a USB stick (falls back to roboRIO storage without one); simulation writes to a local logs/ folder.

Why

The only receiver was NT4Publisher, so every Logger.recordOutput — swerve module states, odometry/pose, turret/shooter/intake/climber, vision — was streamed live to AdvantageScope and never persisted. We have zero .wpilog files from the entire 2026 season; the only surviving telemetry is REVLib's auto-written .revlog CAN logs, which carry no drivetrain or pose data. This one-liner is what was missing.

Notes

  • Matches the AdvantageKit 26 template pattern (WPILOGWriter + NT4Publisher before Logger.start()); no-arg constructor verified against akit-java-26.0.0.
  • CTRE SignalLogger intentionally not enabled: WPILOGWriter covers everything recordOutput produces, and hoot logging risks filling RIO flash when no USB is present.
  • Verified: ./gradlew compileJava BUILD SUCCESSFUL.
  • Before a real session: put a USB stick in the RIO and confirm a log file appears under /U/logs (see PWRDrive/docs/swerve_log_collection.md for the full collection protocol).

🤖 Generated with Claude Code

Until now the only AdvantageKit data receiver was NT4Publisher, so all
telemetry (swerve states, odometry, mechanisms) was streamed live to
AdvantageScope and never written anywhere — no .wpilog files exist from
the entire 2026 season. Add WPILOGWriter so real-robot runs log to
/U/logs on a USB stick (logs/ folder in simulation).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.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.

1 participant