Feature request: Additional fields in the TS log file.

It's all about the code!
Post Reply
spags
contributor
contributor
Posts: 136
Joined: Mon Oct 13, 2014 1:04 am

Feature request: Additional fields in the TS log file.

Post by spags »

Hi,
I've been running the car and logging my runs for later analysis. Specifically so I can write a program to crunch that data to provide tweaks to the map. If and when I finish this it will be GPL'd along with the rest of the project. I'm missing some critical values in the log to do these things, though. Here is a header of an TS log (MSL format). This log was produced by revision 5733.

Code: Select all

"MShift v0.01: Bowling & Grippo MS1 Base Code MShift v0.01"
"Capture Date: Tue Dec 09 19:36:10 AST 2014"
Time    RPM     CLT     IAT     TPS     MAF     MAP     AFR     vBatt   Fuelload        injPulse
        RPM     <B0>C   <B0>C   %       V       MAP     AFR     V       %       ms
172.018 1200    70.0    30.0    0       0.02    8       21.12   7.81    0       1.616

Issues:
- Fuelload is always 0. Probably because I'm using MAF based fueling This should make sense in SD fueling?
- Missing ignition information, maybe ignition advance. This one is really necesary when we start hunting for more power.
- It would be nice if we could break down from the data in the log how much did the base pulse and the enrichment contribute to the final injPulse. This would require at least the enrichments to be added to the log.

Fix:
Not sure how TS works yet, but some of these values are probably already broken out. I looked in firmware/tunerstudio/rusefi.ini and found that there is a DataLog section. Maybe these values can be added here.
User avatar
AndreyB
Site Admin
Posts: 14727
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Feature request: Additional fields in the TS log file.

Post by AndreyB »

I believe

Code: Select all

	float timing = getAdvance(rpm, engineLoad PASS_ENGINE_PARAMETER);
	tsOutputChannels->inj_adv = timing > 360 ? timing - 720 : timing;
adds ignition - this is from a couple days ago.

I've just added 70% of what you were asking for - see https://sourceforge.net/p/rusefi/code/5735/
Can you please try to figure out the rusefi.ini part of the change and email me the newer file or the patch? This would really help :)
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
spags
contributor
contributor
Posts: 136
Joined: Mon Oct 13, 2014 1:04 am

Re: Feature request: Additional fields in the TS log file.

Post by spags »

Ok, it should be in your inbox.
Post Reply