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

Is RL correct for the 3.3V examples? #73

Open
bvernham opened this issue Dec 5, 2023 · 1 comment
Open

Is RL correct for the 3.3V examples? #73

bvernham opened this issue Dec 5, 2023 · 1 comment

Comments

@bvernham
Copy link

bvernham commented Dec 5, 2023

Describe the bug
A clear and concise description of what the bug is.
Not so much a bug but a question
image
In this of the ESP8266 example are you considering RL = 5K? Because the example code is still defaulting to 10K? If the MQ sensor already had a 10K as RL then by adding the voltage divider to "AO" (which is in parallel to "RL" of 10K) aren't you making RL effectively ~3.33K?
To Reproduce
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@berryPiTech
Copy link

Bvernham raises a valid point. To avoid reducing the effective RL, the second voltage divider (for level shifting down from 5v to 3.3v) should employ high impedance.

A 100k/200k combination would be better than the (1.7k/3.3k) combination in the diagram.

I have verified the 100k/200k combination works fine for esp32c3.

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

No branches or pull requests

2 participants