Fix Code Style On common_utils#2526
Merged
Merged
Conversation
rhkoulen
added a commit
that referenced
this pull request
Jun 23, 2026
* wrote the basic framework for exporting common utilities, replaced one such example in offense * comp changes (#2522) * added robot pid values * Final PID values * field teting 3/17 * 03/18fasdfasdfsadjfjsdahflkajsdhfkljasdhfaslfhk * uh shit might work * sdfsadfsfd * march 202sjflsakdjflsadfjlks * defense now has a state for shooting to goal if closest to ball * added bounding checks to ensure kicking field and kick only when ball is slow * comp changes * Fix Code Style On field-testing-3-15 (#2523) automated style fixes Co-authored-by: Squid5678 <Squid5678@users.noreply.github.com> * i forgot header files were a thing * rebase * fix defense shooting * int not double --------- Co-authored-by: rishiso <rishisoni.5678@gmail.com> Co-authored-by: sanat <saisanat@gmail.com> Co-authored-by: Sid Parikh <parikhsid16@gmail.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Squid5678 <Squid5678@users.noreply.github.com> * wrote the basic framework for exporting common utilities, replaced one such example in offense * distilling some computations from offense * refactored solo offense with position utils (also made solo offense into a MVP robot position we can test on sunday) * I think it's bad for me to pollute this namespace? I don't really know how namespaces work. * tiny cleanups * rules * passing a non-positive value into granularity would infinite loop, might as well squash that bug early * Fix Code Style On common_utils (#2526) * automated style fixes * I don't like the way it styled this --------- Co-authored-by: rhkoulen <rhkoulen@users.noreply.github.com> Co-authored-by: r.koulen.p@gmail.com <rkoulen3@gatech.edu> * review fixes * Fix Code Style On common_utils (#2532) automated style fixes Co-authored-by: rhkoulen <rhkoulen@users.noreply.github.com> * fixes to calculate_kick_speed * Fix Code Style On common_utils (#2533) automated style fixes Co-authored-by: rhkoulen <rhkoulen@users.noreply.github.com> --------- Co-authored-by: r.koulen.p@gmail.com <rkoulen3@gatech.edu> Co-authored-by: Aalind Tyagi <42896179+Squid5678@users.noreply.github.com> Co-authored-by: rishiso <rishisoni.5678@gmail.com> Co-authored-by: sanat <saisanat@gmail.com> Co-authored-by: Sid Parikh <parikhsid16@gmail.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Squid5678 <Squid5678@users.noreply.github.com> Co-authored-by: rhkoulen <rhkoulen@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request contains automatic style formatting for PR #2525. @rhkoulen, please merge this branch into your own to pass style checks!