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

Support for VEML6075 and initial Support for VEML7700 #8432

Merged
merged 2 commits into from
May 22, 2020
Merged

Support for VEML6075 and initial Support for VEML7700 #8432

merged 2 commits into from
May 22, 2020

Conversation

device111
Copy link
Contributor

@device111 device111 commented May 12, 2020

Add support for VEML6075.
-Shows UVA, UVB, UVIndex

supported Commands are:

Command Description
VEML6075power Power up/down
VEML6075dynamic HD Mode
VEML6075inttime Integration Time

Add initial Support for VEML7700 (#8260)

  • the implementation based on Adafruit VEML7700 Libary.
  • Shows Illuminance and White Content of Light.

Description:

Related issue (if applicable): fixes #

Checklist:

  • The pull request is done against the latest dev branch
  • Only relevant files were touched
  • Only one feature/fix was added per PR.
  • The code change is tested and works on core ESP8266 V.2.7.1
  • The code change is tested and works on core ESP32 V.1.12.0
  • I accept the CLA.

NOTE: The code change must pass CI tests. Your PR cannot be merged unless tests pass

Add support for VEML6075.
-Shows UVA, UVB, UVIndex

supported Commands are:

| Command        |      Description |
|----------------|------------------|
|VEML6075power   | Power up/down    |
|VEML6075dynamic | HD Mode          |
|VEML6075inttime | Integration Time |

Add initial Support for VEML7700  (#8260)
- the implementation based on Adafruit VEML7700 Libary.
- Shows Illuminance and White Content of Light.
@arendst arendst added the on hold by dev team Result - Feature request put on hold by member of development team label May 13, 2020
@device111
Copy link
Contributor Author

@arendst
are any problems with the PR?

@arendst
Copy link
Owner

arendst commented May 21, 2020

Still investigating the used libraries. I noticed a pow math function using 5k code. AN alternative is available and provided by Tasmota. Needs lib update though. Not a big problem but I need to find time to do it.

@device111
Copy link
Contributor Author

ah ok. thanks for info.
at the moment, the driver don't use "readLuxNormalized" and "readWhiteNormalized" with the math-pow function for the readings.
If i had time, i did a integration without Libary.

@arendst arendst merged commit 2b798ff into arendst:development May 22, 2020
arendst added a commit that referenced this pull request May 22, 2020
- Add support for VEML6075 UVA/UVB/UVINDEX Sensor by device111 (#8432)
- Add support for VEML7700 Ambient light intensity Sensor by device111 (#8432)
@scum2000
Copy link

Guys, wondering if you can help. I'm trying to use both a bme280 and veml6075. I compiled my own tasmota with various versions up to 8.3.1.3. both sensors show up in the i2cscan command but if I comment out #define_bmp in the build the veml6075 shows up in main menu. If I don't bme280(same with bmp280, tried both) shows instead. Do you have any suggestions of anything I can try? Thanks.

@ascillato
Copy link
Contributor

Please, address this to the Tasmota Support Chat. The chat is a better and more dynamic channel for helping you. Github issues are meant for Tasmota Software Bug Reporting.

Please check the Contributing Guideline and Policy and the Support Guide.

Thanks.


Support Information

See Wiki for more information.
See FAQ for common questions/answers and links if none of your question is in the list
See Chat for more user experience.
See Community for forum.
See Code of Conduct

@scum2000
Copy link

Thanks for the response. I already did and they told me to come here. No worries

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on hold by dev team Result - Feature request put on hold by member of development team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants