0000076: Add ability to cell by cell tune tables to memory dump tab - Mantis

Mantis Bugtracker

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

ID

Category

Severity

Reproducibility

Date Submitted

Last Update

0000076

[FreeEMS-Tuner] User Interface

feature

N/A

2008-11-28 23:33

2009-09-04 10:39

Reporter

Fred

View Status

public

Assigned To

sry_not4sale

Priority

normal

Resolution

open

Status

assigned

Product Version

Summary

0000076: Add ability to cell by cell tune tables to memory dump tab

Description

This one should be pretty easy too :-)

You'll need to source the location id from the same drop down.

Then you need three numeric text entry fields :

RPM index
Load index
Value

The three new packet types are :

/* Main table manipulation */
#define adjustMainTableCell 100
#define adjustMainTableRPMAxis 102
#define adjustMainTableLoadAxis 104

The first one has 8 bytes of payload in the form of 4 unsigned shorts.

<start><header> location id, rpm index, load index, cell value <checksum><stop>

The second is :

<start><header> location id, rpm index, rpm value <checksum><stop>

And the third :

<start><header> location id, load index, load value <checksum><stop>

This will form the foundation of tuning into the future :-)

Should be easy as pie to do as well :-) Mean while Aaron can fix those bugs ;-)

Additional Information

Tags

No tags attached.

Issue Type

New Feature

Relationships

Notes
(0000111) Fred (administrator) 2008-11-28 23:33	All yours Ben! :-)

(0000125) BenFenner (developer) 2008-12-02 23:39 edited on: 2008-12-04 14:05	Here's some notes: BenFenner: You want to be able to specify a load and rpm and give it a value, right? Fred: ahh Fred: no Fred: the index of load and rpm BenFenner: The cell-by-cell thing. Fred: so not 5000rpm Fred: but 15th rpm row Fred: you see? BenFenner: Sure. BenFenner: So it should take in two axis coordinates. Fred: yes Fred: and a value BenFenner: And a new value. Fred: yep BenFenner: k BenFenner: And the packet it needs to send out looks like? Fred: 8 bytes long Fred: 4 unsigned shorts Fred: first is location id Fred: you already have a drop down for that Fred: second is rpm index Fred: third load index Fred: fourth value Fred: easy :-) BenFenner: any special header flags, etc? Fred: no, just standard Fred: firmware type and no length or anything

(0000166) sry_not4sale (administrator) 2009-09-04 10:39	New firmware ID's /* Main table manipulation */ #define adjustMainTableCell 300 #define adjustMainTableRPMAxis 302 #define adjustMainTableLoadAxis 304

Issue History
Date Modified	Username	Field	Change
2008-11-28 23:33	Fred	New Issue
2008-11-28 23:33	Fred	Status	new => assigned
2008-11-28 23:33	Fred	Assigned To	=> sry_not4sale
2008-11-28 23:33	Fred	Issue Type	=> New Feature
2008-11-28 23:33	Fred	Note Added: 0000111
2008-11-28 23:33	Fred	Assigned To	sry_not4sale => BenFenner
2008-12-01 19:39	BenFenner	Status	assigned => acknowledged
2008-12-02 23:39	BenFenner	Note Added: 0000125
2008-12-04 14:05	Fred	Note Edited: 0000125
2009-09-04 10:37	sry_not4sale	Status	acknowledged => assigned
2009-09-04 10:37	sry_not4sale	Assigned To	BenFenner => sry_not4sale
2009-09-04 10:38	sry_not4sale	Status	assigned => acknowledged
2009-09-04 10:38	sry_not4sale	Status	acknowledged => assigned
2009-09-04 10:39	sry_not4sale	Note Added: 0000166

Mantis 1.1.7[^]

Copyright © 2000 - 2008 Mantis Group