Skip to content

Add armbot example #70

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

Merged
merged 6 commits into from
Feb 6, 2023
Merged

Add armbot example #70

merged 6 commits into from
Feb 6, 2023

Conversation

megarubber
Copy link
Contributor

Following #49

In constants.py file, the constants are grouped in different classes (I don't know if this style will respect the RobotPy's porting guide).

In robotcontainer.py, I created a function named moveArm, to adapt in a single line (two calls in configureButtonBindings).

@BerkeSinanYetkin
Copy link
Contributor

seems like the first test fails because you haven't used black autoformatter

@megarubber
Copy link
Contributor Author

megarubber commented Feb 6, 2023

seems like the first test fails because you haven't used black autoformatter

oh, it's not finding anything here :(

[@ subsystems]$ python3 -m black drivesubsystem.py 
All done! ✨ 🍰 ✨
1 file left unchanged.

I don't know why this still showing errors here.

@virtuald virtuald merged commit 16bece7 into robotpy:main Feb 6, 2023
@virtuald
Copy link
Member

virtuald commented Feb 6, 2023

black just got updated, and the new black annoyingly removes some blank lines that the old one didn't remove.

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