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
Merged

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 JeffWDH force-pushed the main branch 3 times, most recently from 2646444 to 28b8262 Compare November 1, 2023 12:25
@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.

@JeffWDH JeffWDH force-pushed the main branch 4 times, most recently from c29e8e9 to 34a5dca Compare November 2, 2023 15:06
@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 wled:0_14_1 Nov 2, 2023
softhack007 pushed a commit to MoonModules/WLED-MM that referenced this pull request Nov 11, 2023
Add LDR sensor dusk/dawn preset control Usermod
softhack007 added a commit to MoonModules/WLED-MM that referenced this pull request Dec 15, 2023
Djelibeybi pushed a commit to Djelibeybi/WLED-MM that referenced this pull request Jan 15, 2024
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