Bug found: Aux Sensors & EGO#2

It's all about the code!
Locked
tmbryhn
Posts: 170
Joined: Wed Feb 12, 2020 2:40 am
Location: Norway

Bug found: Aux Sensors & EGO#2

Post by tmbryhn »

Just discovered a bug regarding Aux Linear and WBO2 #2 input while setting up a new basemap on the stimulator.
Custom ECU running latest Proteus F4 release firmware.

I'd like to have 2x EGT inputs that will be connected to Analog Volt 3 and 4.
My EGT module has an output where 0-5V corresponds to 0-1000°C. When I configure this under Aux Sensor diag, the input ranges between 0 - 327, then jumps to (-)327 - 0, and so on as I keep increasing the voltage on the stim pot.
If I define 0-5V = 0-100, it works just fine as intended, ranging from 0-100.

I proceed to disable both Aux Sensor inputs (ADC input = NONE).

When I try to activate EGO #2 on eg. ADC3, which should now be free to use, I don't get any response to Air/Fuel Ratio 2 gauge in TS when I turn the pot on the stim. However, if I re-activate either Aux Linear #1 or #2 and define the ADC input as ADC3 (or ADC4) simultaneous as having ADC3 (or ADC4) mapped to EGO#2, the AFR#2 gauge responds as intended along with the corresponding Aux Linear gauge mapped to the same input channel.

Also, on the Aux Sensors config window, the units for low and high value are noted as (kPa); this might be misleading to some users.

For some reason, I'm unable to attach .msq file in this post.
User avatar
AndreyB
Site Admin
Posts: 14334
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Bug found: Aux Sensors & EGO#2

Post by AndreyB »

Please link a tune and please link a log

What do you call adc3?
Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
tmbryhn
Posts: 170
Joined: Wed Feb 12, 2020 2:40 am
Location: Norway

Re: Bug found: Aux Sensors & EGO#2

Post by tmbryhn »

I just left home for two weeks of work, so I don't have access to either the hardware or msq.

Sorry about the ambiguity; ADC4 = "Analog Volt 4" on proteus.

Is there any way you could make it possible to upload msq files directly as attachments in forum posts?
User avatar
AndreyB
Site Admin
Posts: 14334
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Bug found: Aux Sensors & EGO#2

Post by AndreyB »

tmbryhn wrote:
Thu Jul 20, 2023 11:46 pm
on the Aux Sensors config window, the units for low and high value are noted as (kPa); this might be misleading to some users.
Thank you for your defect report! This issue is now resolved. https://github.com/rusefi/rusefi/commit/c398206f20f8a1922d1c27282fc1c670307b9bb5

I have to insist on one github ticket per issue, this topic seems to three or four separate issues. I am looking to increase my productivity by removing the effort to split between related or not related parts of forum post.
tmbryhn wrote:
Thu Jul 20, 2023 11:46 pm
For some reason, I'm unable to attach .msq file in this post.
rusefi.com/online allows me to be more productive
Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
Locked