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

Add LDR sensor dusk/dawn preset control Usermod #3490

Merged
merged 1 commit into from Nov 2, 2023

Conversation

JeffWDH
Copy link

@JeffWDH JeffWDH commented Oct 28, 2023

This usermod will obtain readings from a Light Dependent Resistor (LDR) and will turn on/off specific presets based on those readings. This is useful for exterior lighting situations where you want the lights to only be on when it is dark out.

Tested on an ESP32 (HUZZAH32) with a generic LDR.

@blazoncek blazoncek changed the base branch from main to 0_14_1 November 1, 2023 09:37
@blazoncek
Copy link
Collaborator

It would be great if you could also update const.h and usermod_list.cpp to allow for simpler compile (compiler -D option).

@JeffWDH
Copy link
Author

JeffWDH commented Nov 1, 2023

It would be great if you could also update const.h and usermod_list.cpp to allow for simpler compile (compiler -D option).

I have made the requested changes.

Copy link
Collaborator

@blazoncek blazoncek left a comment

Choose a reason for hiding this comment

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

Please revert changes to platformio.ini, otherwise the PR is fine.

platformio.ini Outdated Show resolved Hide resolved
@blazoncek
Copy link
Collaborator

Please do not force push changes next time.

@JeffWDH
Copy link
Author

JeffWDH commented Nov 2, 2023

Understood. Other projects I've contributed to specifically ask for all changes to be contained within a single commit, I'll try to remember for next time.

@blazoncek
Copy link
Collaborator

Understood. Other projects I've contributed to specifically ask for all changes to be contained within a single commit, I'll try to remember for next time.

We do have such guidelines as well but they are not strict rules, especially if changes are requested.
It makes reviewing code easier if commits are clear. But you can always branch, squash merge and make a new PR if that may be an issue.

@blazoncek blazoncek merged commit 1543862 into Aircoookie:0_14_1 Nov 2, 2023
12 checks passed
softhack007 pushed a commit to MoonModules/WLED that referenced this pull request Nov 11, 2023
Add LDR sensor dusk/dawn preset control Usermod
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.

None yet

3 participants