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

This plugin is tested and working good. #428

Merged
merged 2 commits into from
Jul 29, 2017

Conversation

tedenda
Copy link
Contributor

@tedenda tedenda commented Jul 10, 2017

@Grovkillen It will now be a standard plugin.

@Grovkillen
Copy link
Member

@tedenda Good, it's a solid plugin! 👍

@JK-de
Copy link
Contributor

JK-de commented Jul 11, 2017

Should be the naming a little bit more descriptive like "Gases - Senseair CO2" ?

@tedenda
Copy link
Contributor Author

tedenda commented Jul 11, 2017 via email

@Grovkillen
Copy link
Member

Grovkillen commented Jul 11, 2017

I think @jkDesignDE idea is great. Lets use that @tedenda
"NDIR CO2 Sensor MH-Z19 [TESTING]" the P049 should also be renamed.

"Gases - MH-Z19 CO2"

@Grovkillen
Copy link
Member

Or is "Gases - CO2 Senseair" and "Gases - CO2 MH-Z19" more generic and place the plugins more alphabetically in the list. More types of gases would otherwise get in between.

@tedenda
Copy link
Contributor Author

tedenda commented Jul 11, 2017 via email

Renamed the plugin as @jkDesignDE and @Grovkillen suggested.
@psy0rz psy0rz merged commit c30074a into letscontrolit:mega Jul 29, 2017
@TD-er
Copy link
Member

TD-er commented Aug 6, 2017

I am using this, but I wouldn't say it is working "good".
Currently the initialisation leaves the "Automatic Base Calibration" (ABC) enabled.
This will lead to reset the base calibration value to the lowest experienced value in the past 24h.
When a room is not ventilated enough in a day, the next day will be completely off.
This ABC should be disabled for normal use and only enabled when enough ventilated (e.g. when lowest value experienced was plausible for good ventilation, or extremely bad calibration value during the day, for example about an hour at near constant value)

I can also make a separate issue for it, when desired.

@Grovkillen
Copy link
Member

What sensor are you using?

@TD-er
Copy link
Member

TD-er commented Aug 6, 2017

MH-Z19 CO2 sensor.
I opened a new issue about it here: #466

@infernix
Copy link
Contributor

infernix commented Aug 6, 2017

Working as intended by manufacturer qualifies as "good" in my book.

@Grovkillen
Copy link
Member

This plugin is referring to the top quality sensor(s) from Senseair. "P052_SenseAir.ino".

The MH-Z19 is a bad version of their S8. The ABC calibration is not comparable whatsoever.

@infernix
Copy link
Contributor

infernix commented Aug 6, 2017

I'm not disagreeing. I'm saying it's up to the user to decide if he uses a high quality or mediocre sensor; when it's working as intended from ESPEasys point of view (e.g. the code is valid) it can leave TESTING, right? So in that sense MH-Z19 should be fine as a standard plugin as well?

@TD-er
Copy link
Member

TD-er commented Aug 6, 2017

@Grovkillen
Oh sorry, I guess I mixed up the threads/commit tabs then.
I was perhaps confused by the relabeling discussion of the MH-Z19 sensor mentioned here also.

Just to be sure... do you have a link to the "not bad version CO2 sensor" this plugin is about? ;)
Never mind, found it... http://www.senseair.com/products/oem-modules/senseair-s8/

@Grovkillen
Copy link
Member

@TD-er good you found it.

@Grovkillen
Copy link
Member

@infernix that is probably right. :) I don't have the MH-Z19 myself but as you say, it is good to have options as a user. I agree that technical wise your plugin is stable and should leave TESTING as well.

@tedenda tedenda deleted the senseair_remove_testing branch August 20, 2017 07:57
@tedenda tedenda restored the senseair_remove_testing branch October 17, 2017 18:06
@tedenda tedenda deleted the senseair_remove_testing branch May 21, 2018 21:27
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

6 participants