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

New implementation of Gun application #16

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

arcadien
Copy link
Owner

Implementation of Gun, using clever components and unit tests.

@arcadien arcadien force-pushed the feat/new_gun_implementation branch from 2dba2f7 to 14307d0 Compare November 24, 2023 09:48
@arcadien arcadien force-pushed the feat/new_gun_implementation branch from 14307d0 to 1cbc09e Compare November 24, 2023 09:48
@arcadien
Copy link
Owner Author

There is a bug in the hardware,

bool Atmega328pHal::buttonIsUp() { return bit_is_clear(PIND, PD3); }

"Button" seems pulled down, not up.

@arcadien arcadien self-assigned this Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant