Merge pull request #79 from miguel5612/codex/modificar-cálculo-usando-voltres-en-readsensor

Fix voltage resolution side effect
This commit is contained in:
Miguel Angel Califa Urquiza 2025-06-04 17:50:20 -05:00 committed by GitHub
commit af291df03d
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -142,14 +142,13 @@ float MQUnifiedsensor::validateEcuation(float ratioInput)
float MQUnifiedsensor::readSensor(bool isMQ303A, float correctionFactor, bool injected)
{
//More explained in: https://jayconsystems.com/blog/understanding-a-gas-sensor
float voltageResolution = _VOLT_RESOLUTION;
float voltRes = _VOLT_RESOLUTION; // preserve global resolution
if(isMQ303A) {
// Issue #42: MQ303A requires a 0.45V offset for the RS calculation. The
// previous implementation permanently decreased _VOLT_RESOLUTION every time
// the sensor was read which produced incorrect results after multiple calls.
voltageResolution -= 0.45;
voltRes = voltRes - 0.45; //Calculations for RS using mq303a sensor look wrong #42
}
_RS_Calc = ((voltageResolution*_RL)/_sensor_volt)-_RL; //Get value of RS in a gas
_RS_Calc = ((voltRes*_RL)/_sensor_volt)-_RL; //Get value of RS in a gas
if(_RS_Calc < 0) _RS_Calc = 0; //No negative values accepted.
if(!injected) _ratio = _RS_Calc / this->_R0; // Get ratio RS_gas/RS_air
_ratio += correctionFactor;