Feature request. Rev limit based on sensors.

Testing and development of Megasquirt 3

Moderators: jsmcortina, muythaibxr

aidandj
Experienced MS/Extra'er
Posts: 255
Joined: Fri Sep 26, 2014 1:38 am

Feature request. Rev limit based on sensors.

Post by aidandj »

Similar to the CLT rev limit. Would it be possible to implement something that could take oil pressure and say shut off the engine below a certain pressure? Or set a rev limit if oil temp goes too high, or AFRs too lean, or EGT too high?

Looking at Limp mode it seems like it is more for malfunctioning sensors.
Matt Cramer
Super MS/Extra'er
Posts: 17507
Joined: Thu Apr 16, 2009 8:08 pm

Re: Feature request. Rev limit based on sensors.

Post by Matt Cramer »

The limp mode can be triggered by sensors malfunctioning OR by the sensor reading exceeding a threshold.
Matt Cramer -1966 Dodge Dart slant six running on MS3X
krisr
Master MS/Extra'er
Posts: 799
Joined: Wed Aug 17, 2005 1:17 am
Location: Sydney, Australia

Re: Feature request. Rev limit based on sensors.

Post by krisr »

Limp mode for any sensor (including generic inputs) would be sweet.

Example: Oil pressure - well... no explanation needed. Coolant pressure - lifted a head, get off the gas ASAP! I'd pay money for these sorts of safety features to work their way into the code as they could save me many thousands down the track.
Sydney, Australia
1971 Holden Monaro HQ
MS3X Sequentially fuelled 400 Pontiac
jsmcortina
Site Admin
Posts: 39618
Joined: Mon May 03, 2004 1:34 am
Location: Birmingham, UK
Contact:

Re: Feature request. Rev limit based on sensors.

Post by jsmcortina »

There's already oil pressure monitoring.

James
I can repair or upgrade Megasquirts in UK. http://www.jamesmurrayengineering.co.uk

My Success story: http://www.msextra.com/forums/viewtopic ... 04&t=34277
MSEXTRA documentation at: http://www.msextra.com/doc/index.html
New users, please read the "Forum Help Page".
krisr
Master MS/Extra'er
Posts: 799
Joined: Wed Aug 17, 2005 1:17 am
Location: Sydney, Australia

Re: Feature request. Rev limit based on sensors.

Post by krisr »

jsmcortina wrote:There's already oil pressure monitoring.
Yep, and i've setup a 0-100PSI transducer input and the D15 with a piezzo buzzer inside the case to trigger as the oil pressure warning output as a noise is sometimes easier to pick-up on than a light BUT to compliment a warning, having the ECU react after a detected failure would IMO be a better approach than to rely on the driver. Same for coolant pressure/lifted cylinder head - hence limp or rev limit based on any sensor input would be a feature i'd really love to see. Not having a go, just some feedback of some practical things i've made notes on as i've been capturing some data from extra sensors i'm datalogging now (TH400 trans line pressure, trans temp, oil temps, etc..).
Sydney, Australia
1971 Holden Monaro HQ
MS3X Sequentially fuelled 400 Pontiac
aidandj
Experienced MS/Extra'er
Posts: 255
Joined: Fri Sep 26, 2014 1:38 am

Re: Feature request. Rev limit based on sensors.

Post by aidandj »

Exactly. A failsafe system that doesn't rely on the driver. Can limp mode be used for generic sensors?

I dug around the code and think I figured out how to implement it. I'll play with it this winter. Are there any timing constraints on the revlimit section of code. I saw it checks the CLT, but is there a limit on how many parameters it can check?
Reverant
Super MS/Extra'er
Posts: 1233
Joined: Sat Apr 15, 2006 12:39 am
Location: Athens, Greece

Re: Feature request. Rev limit based on sensors.

Post by Reverant »

+1 for failsafes based on generic sensors (ie oil pressure, oil temp, trans temp, diff temp, coolant pressure, EGTs, etc).

Right now, we have only 1 actual failsafe - the lean AFR limiter.
The man behind MS Labs
2005 Audi A3 2.0L TFSI DSG AWD - Extreme MS3
2002 Mazda Miata 1.8 6sp - Enhanced MS3 1.4.0, sequential injection, sequential ignition, big turbo, lots of boost
dontz125
Super MS/Extra'er
Posts: 4221
Joined: Mon May 11, 2009 7:14 pm
Location: York, ON
Contact:

Re: Feature request. Rev limit based on sensors.

Post by dontz125 »

The various 'limp' sensors and oil pressure trigger an output pin (if selected). In addition to turning on a lamp, buzzer, air raid siren etc, you could add a transistor to ground out the WBO2 signal, thus tripping the AFR safety system.
Temporarily shut down - back soon!
QuadraMAP Sensor Module -- PWM-to-Stepper Controller -- Dual Coil Driver
Coming soon: OctoMAP Sensor Module
TTR Ignition Systems
aidandj
Experienced MS/Extra'er
Posts: 255
Joined: Fri Sep 26, 2014 1:38 am

Re: Feature request. Rev limit based on sensors.

Post by aidandj »

dontz125 wrote:The various 'limp' sensors and oil pressure trigger an output pin (if selected). In addition to turning on a lamp, buzzer, air raid siren etc, you could add a transistor to ground out the WBO2 signal, thus tripping the AFR safety system.
I could rube goldberg it. But I would rather have a proper software solution.
piledriver
Super MS/Extra'er
Posts: 1681
Joined: Tue Oct 27, 2009 6:24 am
Location: Van Alstyne, Texas

Re: Feature request. Rev limit based on sensors.

Post by piledriver »

aidandj wrote:
dontz125 wrote:The various 'limp' sensors and oil pressure trigger an output pin (if selected). In addition to turning on a lamp, buzzer, air raid siren etc, you could add a transistor to ground out the WBO2 signal, thus tripping the AFR safety system.
I could rube goldberg it. But I would rather have a proper software solution.
I like hardware interlock solutions.
Theres nothing quite like building a crowbar circuit using an actual crowbar.
(Think radar power supply shorting bar)
Always doing things the hard way, MS2 sequential w/ v1.01 mainboard, LS2 coils. 80 mile/day commuter status.
aidandj
Experienced MS/Extra'er
Posts: 255
Joined: Fri Sep 26, 2014 1:38 am

Re: Feature request. Rev limit based on sensors.

Post by aidandj »

Its hijacking another system. The infrastructure is there to add more rev limits.

If the devs don't agree then I will find a hardware solution.
krisr
Master MS/Extra'er
Posts: 799
Joined: Wed Aug 17, 2005 1:17 am
Location: Sydney, Australia

Re: Feature request. Rev limit based on sensors.

Post by krisr »

Hypothetically, how would you do a hardware solution with the MS3Pro? - If you had told me that I needed to crack open a sealed "pro" series ECU to jump wires around to get a safety feature to work, I'll have a Haltech in the car the very next day.

Hardware solutions IMO are bandaids and not very user friendly. Safety features should be in the software and should be scalable and flexible (think like the general purpose outputs) and I think (again, not having a go at anyone) should be on the very top of any TODO list vs the "cool funky race stuff" which may get used by a handful of people, if that. With the IO of these ECU's now plus if you're running an IOX on the CANBUS with an MS3X, we can get so much value out of datalogging the whole drive train but consider the monetary value if it could actually do something about it and prevent a block/gearbox tear up, hence why I saw the value in running trans line pressure sensors, coolant pressure sensors and this thread popping up now is good timing for me.
Sydney, Australia
1971 Holden Monaro HQ
MS3X Sequentially fuelled 400 Pontiac
dontz125
Super MS/Extra'er
Posts: 4221
Joined: Mon May 11, 2009 7:14 pm
Location: York, ON
Contact:

Re: Feature request. Rev limit based on sensors.

Post by dontz125 »

krisr wrote:Hypothetically, how would you do a hardware solution with the MS3Pro?
Both the AFR signal and the warning signal are available at the wiring. The entire point of the warning signal is that it leaves the box and travels to the dash; since the vast majority of the MS3-Pro's outputs are ground-switching, a simple Schottky diode from the WBO2 wire to the warning signal wire would do the job nicely. It would also be easy to remove if so desired when / if this function is added to the code.
Temporarily shut down - back soon!
QuadraMAP Sensor Module -- PWM-to-Stepper Controller -- Dual Coil Driver
Coming soon: OctoMAP Sensor Module
TTR Ignition Systems
piledriver
Super MS/Extra'er
Posts: 1681
Joined: Tue Oct 27, 2009 6:24 am
Location: Van Alstyne, Texas

Re: Feature request. Rev limit based on sensors.

Post by piledriver »

Hardware interlocks are what you should (really, MUST) use if actual safety is involved.

Software is great, but sensors can lie and code can have ...unintended features.

Overpressure or overtemp switches that literally are designed to blow the fuse (crowbar) in event of a failure are examples.
You probably don't want that when crossing a busy intersection or merging onto a freeway though.

Software solutions are never considered enough, require extensive proof and redundancy as to actual safety items.
Always doing things the hard way, MS2 sequential w/ v1.01 mainboard, LS2 coils. 80 mile/day commuter status.
dontz125
Super MS/Extra'er
Posts: 4221
Joined: Mon May 11, 2009 7:14 pm
Location: York, ON
Contact:

Re: Feature request. Rev limit based on sensors.

Post by dontz125 »

For a hard engine cut, yes - it's hard to beat hardware. But I thought the topic was to limit operation if something dodgy is detected; how is that done -other- than code? What fuses would be crowbarred by a coolant pressure switch upon detecting (what is probably) a blown head gasket?

(Fuse that releases a solenoid-triggered throttle stop that limits opening, and cuts out the boost solenoid? Another fuse that cuts out half the spark plugs?)
Temporarily shut down - back soon!
QuadraMAP Sensor Module -- PWM-to-Stepper Controller -- Dual Coil Driver
Coming soon: OctoMAP Sensor Module
TTR Ignition Systems
krisr
Master MS/Extra'er
Posts: 799
Joined: Wed Aug 17, 2005 1:17 am
Location: Sydney, Australia

Re: Feature request. Rev limit based on sensors.

Post by krisr »

dontz125 wrote:Both the AFR signal and the warning signal are available at the wiring. The entire point of the warning signal is that it leaves the box and travels to the dash; since the vast majority of the MS3-Pro's outputs are ground-switching, a simple Schottky diode from the WBO2 wire to the warning signal wire would do the job nicely. It would also be easy to remove if so desired when / if this function is added to the code.
Ahh yeah that's true. But then you'd need to replicate the warning signal (general purpose output?) per input sensor condition you'd want an action for, which is where it's not user-friendly and is a combination of extra hardware and software plus in some cases you wouldn't need or want to completely cut the engine out especially on a freeway @ 60mph or in front of another vehicle that may cause a side impact. The question of sensor reliability is still called into question (we're talking any sensor, not just one tapped into the oil pressure line) for either hardware or software but the action from software can potentially deliver a more desired outcome if configured that way, i.e. have the ability to have it rev limit the motor in order to nurse the car/bike to the side of the road/breakdown bay.

I just feel (strongly) that this discussion around safety could really help someone or all users. Even though software safety may not be considered as "enough", it's still better than nothing when you're looking outside of AFR. Extending the AFR safety feature to be a more generic safety feature for any input condition coupled with the work already done for CEL flashcodes/warning outputs/etc might be a good platform to consolidate the existing work into 1 theme, i'm just struggling to understand the resistance? Again, not having a go but trying to keep the discussion flowing :RTFM:
Sydney, Australia
1971 Holden Monaro HQ
MS3X Sequentially fuelled 400 Pontiac
piledriver
Super MS/Extra'er
Posts: 1681
Joined: Tue Oct 27, 2009 6:24 am
Location: Van Alstyne, Texas

Re: Feature request. Rev limit based on sensors.

Post by piledriver »

Resistance is probably too strong a word, it's closer to "inertia".
Mostly we are just trying to get you where you want to be, now, with what you have.

My HW interlock rant was more or less in response to
Safety features should be in the software and should be scalable and flexible (think like the general purpose outputs)
No. They. Shouldn't. Not real "safety" features.

Very few folks actually contribute code, so those that do get to choose, and they have plenty of "external input" constantly.

Picking and choosing what they want to tackle next is probably a major drag.
Always doing things the hard way, MS2 sequential w/ v1.01 mainboard, LS2 coils. 80 mile/day commuter status.
dontz125
Super MS/Extra'er
Posts: 4221
Joined: Mon May 11, 2009 7:14 pm
Location: York, ON
Contact:

Re: Feature request. Rev limit based on sensors.

Post by dontz125 »

The problem and differences of opinion I think comes in with varying definitions of "safety". Theoretically, any critical engine malfunction would trip the safety system to hard-cut the engine *NOW*, preventing the con-rods from taking a constitutional around the inside of your engine bay. In the Real World (tm), there are often circumstances where eating a valve is preferable to, say, eating an 18-wheeler. Leaving the engine running, and possibly inflicting $$ in damage to the internals, may as noted in an earlier post be the difference between pulling off to the side of a busy expressway and becoming a chicane in the middle of that highway. I once had the engine in my old POS K-car conk out on the 401 in heavy Toronto traffic; gliding off to the shoulder with increasingly alarming speed differentials was NOT fun ...
Temporarily shut down - back soon!
QuadraMAP Sensor Module -- PWM-to-Stepper Controller -- Dual Coil Driver
Coming soon: OctoMAP Sensor Module
TTR Ignition Systems
Reverant
Super MS/Extra'er
Posts: 1233
Joined: Sat Apr 15, 2006 12:39 am
Location: Athens, Greece

Re: Feature request. Rev limit based on sensors.

Post by Reverant »

So ejecting a rod, spilling all the oil and coolant on the road is safe, but shutting down the engine is not?

Losing oil pressure or overheating to the point where the pistons seize in the cylinder *with the car in motion* is safe but shutting down the engine before that happens is not?

And of course, these ECUs are never installed on road cars, right? They are only for use on off-road vehicles. :roll:

Engine safety is not just for engine safety alone. Shutting down the engine before the damage happens will still allow you to use the brakes and steering, etc.
The man behind MS Labs
2005 Audi A3 2.0L TFSI DSG AWD - Extreme MS3
2002 Mazda Miata 1.8 6sp - Enhanced MS3 1.4.0, sequential injection, sequential ignition, big turbo, lots of boost
piledriver
Super MS/Extra'er
Posts: 1681
Joined: Tue Oct 27, 2009 6:24 am
Location: Van Alstyne, Texas

Re: Feature request. Rev limit based on sensors.

Post by piledriver »

Reverant wrote:So ejecting a rod, spilling all the oil and coolant on the road is safe, but shutting down the engine is not?

Losing oil pressure or overheating to the point where the pistons seize in the cylinder *with the car in motion* is safe but shutting down the engine before that happens is not?

And of course, these ECUs are never installed on road cars, right? They are only for use on off-road vehicles. :roll:

Engine safety is not just for engine safety alone. Shutting down the engine before the damage happens will still allow you to use the brakes and steering, etc.
Sometimes it will be.
Sometimes it won't.

You might want to get outta the way of the train or semi first.

Volunteering to write the code for that godlike software decision that determines who is the windscreen or bug, and when, that must work correctly, always?

The existing firmware allows you to make that call already, (in a great many cases, memory has hard limits for adding options) as described.
You can shut things down as hard as you like, via several methods.
Always doing things the hard way, MS2 sequential w/ v1.01 mainboard, LS2 coils. 80 mile/day commuter status.
Post Reply