8000 fix: remove debugging printout for snapshot methods by samfreund · Pull Request #1823 · PhotonVision/photonvision · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: remove debugging printout for snapshot methods #1823

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 10 commits into from
Mar 19, 2025
Merged

fix: remove debugging printout for snapshot methods #1823

merged 10 commits into from
Mar 19, 2025

Conversation

samfreund
Copy link
Member
@samfreund samfreund commented Mar 19, 2025 •
edited
Loading

Description

fixes excess debugging messages

Meta

Merge checklist:

  • Pull Request title is short, imperative summary of proposed changes
  • The description documents the what and why
  • If this PR changes behavior or adds a feature, user documentation is updated
  • If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly
  • If this PR touches configuration, this is backwards compatible with settings back to v2024.3.1
  • If this PR addresses a bug, a regression test for it is added

@samfreund samfreund requested a review from a team as a code owner March 19, 2025 19:05
@mcm001 mcm001 merged commit 002373e into PhotonVision:main Mar 19, 2025
36 checks passed
@samfreund samfreund deleted the fix-remove-debugging-printout branch March 19, 2025 19:54
@mcm001 mcm001 mentioned this pull request Mar 25, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mcm001 mcm001 mcm001 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@samfreund @mcm001
0