Skip to content

High fidelity vision sim per Issue 197#227

Open
koolpoolo wants to merge 4 commits intomainfrom
high-fidelity-vision-sim
Open

High fidelity vision sim per Issue 197#227
koolpoolo wants to merge 4 commits intomainfrom
high-fidelity-vision-sim

Conversation

@koolpoolo
Copy link
Copy Markdown
Contributor

just run ./gradlew simulateJava -PhighFidelityVision=true
to turn on higher res

@koolpoolo
Copy link
Copy Markdown
Contributor Author

#197

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a high-fidelity vision simulation mode, configurable via a Gradle property. The changes include passing the 'highFidelityVision' flag to the simulation JVM and adjusting camera resolution and minimum target area constants accordingly. A review comment suggests refining the 'kMinTargetAreaPixels' value for high-fidelity mode to 133.0 to ensure it scales proportionally with the resolution increase, maintaining consistency across simulation modes.

Comment thread src/main/java/frc/robot/sim/visionproducers/VisionSimConstants.java Outdated
@koolpoolo koolpoolo changed the title High fidelity vision sim per Issue-197 High fidelity vision sim per Issue 197 Apr 15, 2026
@koolpoolo koolpoolo requested a review from jamesdooley4 April 15, 2026 00:06
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@koolpoolo koolpoolo linked an issue Apr 15, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Member

@jamesdooley4 jamesdooley4 left a comment

Choose a reason for hiding this comment

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

Also need to update VisionSim to change the estStdDev calculation based on the low fidelity / high fidelity switch. See how they were changed when the low-fidelity change was made

Comment thread src/main/java/frc/robot/sim/visionproducers/VisionSimConstants.java Outdated
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.

Change limelight res in sim

2 participants