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

Added some features #25

Merged
merged 21 commits into from
Nov 22, 2023
Merged

Conversation

SupremeSports
Copy link
Contributor

Instead of hardcoding values, user can change them in definition/setup

The extra features of this request:

  1. setSyncDir(uint8_t dir) to adjust the interrupt direction FALLING, RISING, CHANGE
  2. setSyncPullup(bool pullup) to activate internal pull-up resistor on zero-crossing detection
  3. setGateTurnOffTime(uint16_t timeOff) to change the value of the gate turn off time

Added extra keywords for new functions
Added new functions
- setSyncDir(uint8_t dir) to adjust the interrupt direction FALLING, RISING, CHANGE
- setSyncPullup(bool pullup) to activate internal pull-up resistor on zero-crossing detection
- setGateTurnOffTime(uint16_t timeOff) to change the value of the gate turn off time
Added new functions
- setSyncDir(uint8_t dir) to adjust the interrupt direction FALLING, RISING, CHANGE
- setSyncPullup(bool pullup) to activate internal pull-up resistor on zero-crossing detection
- setGateTurnOffTime(uint16_t timeOff) to change the value of the gate turn off time
Added new functions
- setSyncDir(uint8_t dir) to adjust the interrupt direction FALLING, RISING, CHANGE
- setSyncPullup(bool pullup) to activate internal pull-up resistor on zero-crossing detection
- setGateTurnOffTime(uint16_t timeOff) to change the value of the gate turn off time
Added new functions
- setSyncDir(uint8_t dir) to adjust the interrupt direction FALLING, RISING, CHANGE
- setSyncPullup(bool pullup) to activate internal pull-up resistor on zero-crossing detection
- setGateTurnOffTime(uint16_t timeOff) to change the value of the gate turn off time
Explain new functions
@SupremeSports SupremeSports marked this pull request as ready for review September 8, 2021 16:23
@bcelary
Copy link

bcelary commented Mar 27, 2023

I am wondering why this couldn't be accepted. Are there any issues with a different edge direction of the interrupt or having an internal pull up resistor enabled? I am using some optocoupler that forces me to either mess with soldering pull down resistor or simply change to FALLING and enable pull up resistor to make it easier. Wondering if I could simply use this patch 🤔

@SupremeSports
Copy link
Contributor Author

SupremeSports commented Mar 27, 2023

I am wondering why this couldn't be accepted. Are there any issues with a different edge direction of the interrupt or having an internal pull up resistor enabled? I am using some optocoupler that forces me to either mess with soldering pull down resistor or simply change to FALLING and enable pull up resistor to make it easier. Wondering if I could simply use this patch 🤔

I had the same optocoupler issue, that's why I did this update and gave it to them but they won't accept it

It works flawlessly on my project; FALLING with internal pullups

@bcelary
Copy link

bcelary commented Mar 27, 2023

Thanks. I have already forked the repo to change the maximum dimmers, so might as well use this pull request. I am using Arduino Mega. Will see how it goes.

@fabianoriccardi
Copy link
Owner

Sorry, I missed this update. I gave it a quick glance and it seems fine. I will review it as soon as possible!

adisbladis and others added 9 commits November 5, 2023 16:41
This adds support for RP2040 based boards such as the Pico and the
Pico W using arduino-pico.

Note that this is distinctly different from the mbed-OS based platform which would have to be implemented separately.

I have tested this on a Pico W.
@fabianoriccardi fabianoriccardi merged commit 3137e9e into fabianoriccardi:main Nov 22, 2023
7 checks passed
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.

4 participants