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

Manager #5

Merged
merged 11 commits into from
Dec 29, 2024
Merged

Manager #5

merged 11 commits into from
Dec 29, 2024

Conversation

PotmanNob
Copy link
Contributor

w manager trust

@nkalupahana
Copy link
Member

nkalupahana commented Dec 29, 2024

No idea how much free time y'all have, but it would be so cool to be able to take a subsystem's trigger setup and export a state machine diagram for it to make it easier to visualize. You could have it output some sort of JSON object on init with the different triggers (to SmartDashboard or to the console or to a file or whatever) (put it in the Subsystem superclass), and then pass that to a pretty simple Python script? ChatGPT could def do most of the work. Would highly recommend, bc this is pretty complex and difficult to parse just by reading the code, and it's also hard to make sure you didn't miss anything

Copy link
Collaborator

@GreenTomato5 GreenTomato5 left a comment

Choose a reason for hiding this comment

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

U wana do that diagraming thing? or someone else @PotmanNob

src/main/java/frc/robot/subsystems/Manager/Manager.java Outdated Show resolved Hide resolved
src/main/java/frc/robot/subsystems/Manager/Manager.java Outdated Show resolved Hide resolved
src/main/java/frc/robot/subsystems/Manager/Manager.java Outdated Show resolved Hide resolved
Copy link
Collaborator

@GreenTomato5 GreenTomato5 left a comment

Choose a reason for hiding this comment

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

Yeah this looks chill if it works in sim

src/main/java/frc/robot/subsystems/Manager/Manager.java Outdated Show resolved Hide resolved
src/main/java/frc/robot/subsystems/Manager/Manager.java Outdated Show resolved Hide resolved
Copy link
Collaborator

@GreenTomato5 GreenTomato5 left a comment

Choose a reason for hiding this comment

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

🥇 💯 🎂

@PotmanNob PotmanNob merged commit 70a92af into main Dec 29, 2024
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.

3 participants