If this is really a debug feature (as the code shows), that constraint should not be an issue. If you want it to be a feature used for normal operation then it is a different thing.
I agree with you but this is obviously not what is intended by that piece of code because it actually repurposes sensor variables for some debug information. That is not how normal features work so I assume this is due to the current limitations, as you mentioned.
Still, as the knock code only retards cylinders that are up to ram4.no_cyl, there's no point in recording for cylinders that are never going to be retarded. Just saying.
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
The problem is that you're looking at it as a normal logging feature which it is not. It's a debug feature. It will overwrite normal outputs which makes it unacceptable as a normal feature regardless of the number of wasted outputs.
It would have to clearly prevent the use of the overwritten outputs (in the ini, presumably) and have information in the tooltips about this to even start to be considered as a normally usable logging feature.
Yea if you are controlling fuel per cylinder and spark per cylinder you should be able to see what each is doing on a per cylinder basis.
You should also be able to see knock on a per cylinder basis. This is common sense for knock control.
I understand that re-writing the code to overcome the 512byte outpc limit is a task that will consume a lot of man hours, so as a workaround, PLEASE just use the no_cyl variable instead of 16. That way we can at least log each cylinder without losing all generic sensors, at least until we overcome the outpc limit. It's a 10 second code change. I just don't see why one of the devs can't do it.
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