main.qml

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-12262: INT: FW Support (Reset Diagnostic Data)

  1. … 12 more files in changeset.
remove the empty line. lower the column alignment spaces.

remove the empty line.
lower the column alignment spaces.

please align & empty line

please align
& empty line

Please align remove the empty line.

Please align
remove the empty line.

remove empty

remove empty

remove empty line.

remove empty line.

Remove empty line.

Remove empty line.

remove Data, and fix infoText in doc.

remove Data, and fix infoText in doc.

Fix #9 to #10

Fix #9 to #10

remove Data, and fix infoText in doc.

remove Data, and fix infoText in doc.

fix the Src, and CAN ID.

fix the Src, and CAN ID.

Fix the Src, Should be HD. *Fix the CAN ID, should be 0x040
  • Fix the Src, Should be HD.
  • Fix the CAN ID, should be 0x040
the "Data" is redundant. in the log, there is a section for the type if is data will be ",D," Please, update all the other models and the "| infoText|" in the header documents as well.

the "Data" is redundant.
in the log, there is a section for the type if is data will be ",D,"
Please, update all the other models and the "| infoText|" in the header documents as well.

There is a comment in the model. I that changes please don't forget to fix this as well.

There is a comment in the model.
I that changes please don't forget to fix this as well.

This seems a big type for just a boolean value to me. Please check the FW code as well.

This seems a big type for just a boolean value to me.
Please check the FW code as well.

remove the empty line.

remove the empty line.

Don't see why the CRev is necessary? Also, formatting doesn't make sense to me. fpgaID is in between HD and FPGA versions !!! I think it should be like: v<hdMajor(%1)>.<hdMinor(%2)>.<hdMicro(%3)>...

Don't see why the CRev is necessary?

Also, formatting doesn't make sense to me.
fpgaID is in between HD and FPGA versions !!!

I think it should be like:
v<hdMajor(%1)>.<hdMinor(%2)>.<hdMicro(%3)>.<hdBuild(%4)><' '>v<fpgaID(%5)>.<fpgaMajor(%6)>.<fpgaMinor(%7)>.<fpgaLab(%8)><' - '><compRev(%9)>

or move the fpgaID at the end of the FPGA versions.

The message is a U16 and each valve is a bit, and it's not clear to me why the type has been changed from bool to a very big uint32 type? Although it was defined as boolean, qml was showing it as a...

The message is a U16 and each valve is a bit, and it's not clear to me why the type has been changed from bool to a very big uint32 type?
Although it was defined as boolean, qml was showing it as an int value, so what are we going to achieve here?

HD Accelerometer is not being used anymore? If so we should put a TODO comment to remove the model/view later to keep the Application lighter.

HD Accelerometer is not being used anymore?
If so we should put a TODO comment to remove the model/view later to keep the Application lighter.

DG Accelerometer is not being used anymore? If so we should put a TODO comment to remove the model/view later to keep the Application lighter.

DG Accelerometer is not being used anymore?
If so we should put a TODO comment to remove the model/view later to keep the Application lighter.

please remove the empty line.

please remove the empty line.

Please remove the extra empty line.

Please remove the extra empty line.

Same as hd version please consult with the users of this information.

Same as hd version
please consult with the users of this information.

RESOLVED. The Application will not use that new parameter though.

RESOLVED.
The Application will not use that new parameter though.

Let's keep it.

Let's keep it.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.

RESOLVED.