Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-12262: INT: Code reviews wrap up & support

- Addressed all the code review comments in another branch in here to facilitate the code review process and make the code ready while the actual code owner were busy supporting system for demo.

- Added another condition to the Alarm top/right countdown to hide the timer when FW is not even in the Treatment mode by the HDOpMode.

    • -6
    • +6
    ./hd/data/treatment/MTreatmentSalineData.h
    • -4
    • +4
    ./hd/data/treatment/MTreatmentStopData.h
  1. … 18 more files in changeset.
DEN-12262: INT: Code reviews wrap up & support

- Addressed all the code review comments in another branch in here to facilitate the code review process and make the code ready while the actual code owner were busy supporting system for demo.

- Added another condition to the Alarm top/right countdown to hide the timer when FW is not even in the Treatment mode by the HDOpMode.

    • -6
    • +6
    ./hd/data/treatment/MTreatmentSalineData.h
  1. … 19 more files in changeset.
Bamboo Commit: Updated MAlarmMapping.cpp

DEN-12262: INT: Log Dialin Messages

- Log Dialin messages as part of the unhandled messages.

    • -6
    • +6
    ./dg/data/post/MDGPostFinalResultData.h
    • -6
    • +6
    ./dg/data/post/MDGPostSingleResultData.h
    • -6
    • +6
    ./hd/data/post/MHDPostFinalResultData.h
    • -3
    • +3
    ./hd/data/post/MHDPostSingleResultData.h
  1. … 6 more files in changeset.
DEN-12262: INT: FW Support (Reset Diagnostic Data)

- Reorganized the Diagnostics screen a little more.

  1. … 3 more files in changeset.
DEN-12262: INT: FW Support (Reset Diagnostic Data)

  1. … 12 more files in changeset.
Bamboo Commit: Updated MAlarmMapping.cpp

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.