Isn't rawResultCopy always positive here ?

[lahirunirmalx]

it seems here doing some thing . is rawResultCopy == -32768 ever going to happen after abs() ?
all the other places you have done
int16_t rawResult = readRegister(ADS1115_CONV_REG);

void ADS1115_WE::setAutoRange(){
    setVoltageRange_mV(ADS1115_RANGE_6144);
    
    if(deviceMeasureMode == ADS1115_SINGLE){
        startSingleMeasurement();
        while(isBusy()){delay(0);}
    }
    // 
 int16_t rawResult = abs(readRegister(ADS1115_CONV_REG)); // here ?
    int16_t rawResultCopy = rawResult;
    if(rawResultCopy == -32768){
        rawResultCopy++; 
    }
    rawResultCopy = abs(rawResultCopy);

[wollewald]

Hi @lahirunirmalx , thanks. Stupid bug! I will change that. I am going to change also few other things and will do that all together in one new release. So, might take few days.

[lahirunirmalx]

I didn't fully understand the code . also i'm not that good with C type system
feel like need to use static casting to int16_t then do the operation rather than use abs()
that's why I didn't put a PR

[wollewald]

Hi @lahirunirmalx, would be cleaner to change it explicitely to int16_t but it's also working like this. In case the value is -32768, 1 needs to be added because otherwise abs(rawValue) is not working.
I have put the abs() function after adding 1. And I simplified the code. No rawResultCopy was needed.

The bigger change in the new release is that now the ranges of the channels can be saved (see AutoRange.ino).