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

Vision #34

Closed
wants to merge 33 commits into from
Closed

Vision #34

wants to merge 33 commits into from

Conversation

Hannahjjj97
Copy link
Contributor

limelight 和 PhotonVision 掃 AprilTag
測試過了

Hannahjjj97 and others added 21 commits January 26, 2025 03:12
新增了一些定義
add getTx() getTy() getTa()
move "ClimberSubsystem.java" into package "subsystems"
4 space indent ->  2 space indent
Unify first "final" and then "static"
Unify first "static" and then "final"
remove unused import
remove unused import
setsetpoint -> setSetpoint
climbermotor -> climberMotor
add photonlib.json
add Steaming.java
add PhotonCamera camera
add startAutomaticCapture()
delete Steaming.java
copy the file and fix
add TagTrackingPhotonVIsion.java
@Hannahjjj97 Hannahjjj97 requested a review from kennhung February 5, 2025 07:49
@Hannahjjj97 Hannahjjj97 enabled auto-merge February 5, 2025 07:54
some modification to pass styleCheck
some modifications to pass styleCheck
change the package name to pass styleCheck
change the order of import
change the order of import
change indentation level 4 to level 2
Change the member name ID to id
Some member names have not been changed.
Some modifications to pass styleCheck
Some names have not been changed.
Change the order of import to pass styleCheck
Copy link
Member

@kennhung kennhung left a comment

Choose a reason for hiding this comment

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

Vision 資料夾要改小寫
macOS、Windows 沒辦法改,可以用 GitHub 線上編輯器

@Hannahjjj97 Hannahjjj97 closed this Feb 5, 2025
auto-merge was automatically disabled February 5, 2025 13:59

Pull request was closed

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.

4 participants