Skip to content

sim working and merged with main, but constants are sus#80

Open
rhit-collinkn wants to merge 9 commits intomainfrom
temp-sim-fixes
Open

sim working and merged with main, but constants are sus#80
rhit-collinkn wants to merge 9 commits intomainfrom
temp-sim-fixes

Conversation

@rhit-collinkn
Copy link
Contributor

No description provided.

@rhit-collinkn
Copy link
Contributor Author

@kt-h and @shlap I realized I forgot to push this code earlier, but this should work to test your code with sim at least for intake and shooter, but I recommend running a quick systems check on the real bot before merging into main. I added checks for if robot is real to have different constants for sim and real, and modified a couple W8 sim files to get runPosition working in sim, so it shouldn't affect anything on the real bot, but double check in case I made a mistake. Let me know if you need any help with anything

Also @kt-h I'm guessing you've already checked this, but Garrett recommended maybe checking limelight pose relative to robot. He had similar behavior with MT1 working but MT2 not working when the camera pose was flipped (I think position was correct but rotation was not? Not sure, feel free to ping him if you want any more info)

@kt-h
Copy link

kt-h commented Mar 19, 2026

@rhit-collinkn Ooh, interesting. Thank you (and Garrett)! I will double-check the Limelight orientation.

@rhit-collinkn
Copy link
Contributor Author

Branch is now updated with changes from fixing-accidental-deletions. Issues occurred from merging PR that had old code remaining, very sorry about that

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