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

Custom logger #49

Merged
merged 4 commits into from
Jun 19, 2024
Merged

Custom logger #49

merged 4 commits into from
Jun 19, 2024

Conversation

00magikarp
Copy link
Member

No more using Logger.recordOutput, now does DebugLogger.recordOutput and DebugLogger.recordDebugOutput!! 🤩🤩

@00magikarp 00magikarp requested a review from sswadkar June 19, 2024 03:53
Copy link
Member

@sswadkar sswadkar left a comment

Choose a reason for hiding this comment

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

rename to CustomLogger other than that everything looks good

@00magikarp
Copy link
Member Author

made changes and works in sim 👍 👍

also added processInputs lmk if not necessary

inline fun processInputs(key: String, inputs: LoggableInputs) {
Logger.processInputs(key, inputs)
}

@00magikarp 00magikarp merged commit a07407f into worlds-2024 Jun 19, 2024
1 check passed
@00magikarp 00magikarp deleted the custom-logger branch June 19, 2024 15:59
github-merge-queue bot pushed a commit that referenced this pull request Jun 19, 2024
* worlds load in debugging

* worlds q1-19

* code day 1

* adjust amp center line without first note

* fried at worlds but its over?

* tune swerve turning

* Custom logger (#49)

* add recordOutput method to DebugLogger

* rename to CustomLogger

* fix photonlib.json (now builds)

* add processInputs as a method under CustomLogger

---------

Co-authored-by: Matthew Choulas <[email protected]>
Co-authored-by: AlphaPranav9102 <[email protected]>
Co-authored-by: 00magikarp <[email protected]>
@00magikarp 00magikarp restored the custom-logger branch June 20, 2024 00:06
@00magikarp 00magikarp deleted the custom-logger branch June 20, 2024 00:07
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