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.

Sign up for GitHub

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

Jump to bottom

Dylan Masulis onboarding #58

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Dylan Masulis onboarding #58

wants to merge 1 commit into from

Conversation

dylanmas
Copy link
Collaborator

@dylanmas dylanmas commented Oct 6, 2024

  • Completed schematic + layout for the LED Circuit project.
  • Completed the Heartbeat project.
  • Completed the Potentiometer Blink Rate project.
    • Defined AnalogIn interface to read from the potentiometer pin.
    • Because AnalogIn returns a float from 0.0 to 1.0, it multiplies an existing number to modify the tickThreshold interval.
    • tickThreshold interval is a custom variable that varies based on the analog input from the potentiometer.

@dylanmas dylanmas self-assigned this Oct 6, 2024
@Suneetr2
Copy link
Contributor

Suneetr2 commented Oct 6, 2024

LED project good
LED blinky good
pretty good approach to challenge FW project, looks good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yummy-tofu yummy-tofu Awaiting requested review from yummy-tofu

@Suneetr2 Suneetr2 Awaiting requested review from Suneetr2

At least 2 approving reviews are required to merge this pull request.

Assignees

@dylanmas dylanmas

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dylanmas @Suneetr2