application

Clone Tools

Stats

Commits this week: 6
Total Committers: 14
Last Commit: 20 Mar
Commits this week: 6

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 04 May 2021
7:54 pm

Behrouz NematiPour

DEN-8316: TxLog: Implementation [Simulator/Testing]

Options
    • -1
    • +1
    /denali.pro.user
    • -0
    • +4
    /en_US.udic
    • -0
    • +2
    /resources/settings/Instructions/Instructions.conf
    • binary
    /resources/settings/Instructions/water sampling_v1.png
    • -1
    • +9
    /sources/storage/DriveWatcher.cpp
    • -4
    • +4
    /sources/storage/StorageGlobals.cpp
3:44 pm

plucia  (deleted user)

Now updated to just variables since they are not used in qml

Now updated to just variables since they are not used in qml

3:44 pm

plucia  (deleted user)

The m is now removed and replaced with underscores to indicate that they are private.

The m is now removed and replaced with underscores to indicate that they are private.

3:43 pm

plucia  (deleted user)

Done

Done

3:43 pm

plucia  (deleted user)

Done

Done

3:43 pm

plucia  (deleted user)

Done

Done

3:43 pm

plucia  (deleted user)

Done

Done

3:43 pm

plucia  (deleted user)

Done

Done

3:43 pm

plucia  (deleted user)

Done

Done

3:43 pm

Peter Lucia <plucia@diality.com>

DEN-7756: Addressed code review comments

Options
    • -5
    • +5
    /denali.pro
    • -3
    • +3
    /sources/gui/qml/components/InformationTable.qml
    • -2
    • +2
    /sources/gui/qml/components/InformationTableEntry.qml
    • -2
    • +2
    /sources/gui/qml/globals/Fonts.qml
    • -2
    • +2
    /sources/gui/qml/pages/pretreatment/create/PreTreatmentUltrafiltration.qml
    • -22
    • +0
    /sources/model/MDGSerialNumberResponse.cpp
    • -64
    • +0
    /sources/model/MDGSerialNumberResponse.h
    • -21
    • +0
    /sources/model/MHDSerialNumberResponse.cpp
    • -64
    • +0
    /sources/model/MHDSerialNumberResponse.h
    • -2
    • +2
    /sources/model/dg/adjustment/MAdjustDGServiceResponse.h
    • -0
    • +22
    /sources/model/dg/adjustment/MDGSerialNumberResponse.cpp
    • -0
    • +64
    /sources/model/dg/adjustment/MDGSerialNumberResponse.h
    • -0
    • +21
    /sources/model/hd/adjustment/MHDSerialNumberResponse.cpp
    • -0
    • +64
    /sources/model/hd/adjustment/MHDSerialNumberResponse.h
    • -37
    • +37
    /sources/view/VDeviceInformation.cpp
    • -14
    • +12
    /sources/view/VDeviceInformation.h
3:28 pm

plucia  (deleted user)

This is used in INTERPRET_RECVD_MT_MESSAGE in MessageInterpreter.cpp. It is overridden so a broadcast is sent out instead of a direct message to the HD.

This is used in INTERPRET_RECVD_MT_MESSAGE in MessageInterpreter.cpp.
It is overridden so a broadcast is sent out instead of a direct message to the HD.

2:54 pm

Peter Lucia <plucia@diality.com>

DEN-8064: Addressed code review comments

Options
    • -1
    • +1
    /scripts/setup.sh
2:53 pm

plucia  (deleted user)

Done

Done

2:49 pm

Peter Lucia <plucia@diality.com>

DEN-8064: Addressed code review comments

Options
    • -2
    • +0
    /denali.pro
    • -1
    • +0
    /sources/gui/GuiGlobals.h
    • -5
    • +0
    /sources/model/MModel.h
    • -39
    • +0
    /sources/model/hd/data/MAlarmVolumeData.cpp
    • -74
    • +0
    /sources/model/hd/data/MAlarmVolumeData.h
    • -1
    • +1
    /sources/view/VBrightness.cpp
2:49 pm

plucia  (deleted user)

Fixed

Fixed

2:49 pm

plucia  (deleted user)

Done

Done

1:06 pm

plucia  (deleted user)

That works too. No need for tee here. I've updated it

That works too. No need for tee here. I've updated it

1:06 pm

Peter Lucia <plucia@diality.com>

DEN-8064: Addressed code review comments

Options
    • -1
    • +1
    /scripts/brightness_set.sh
    • -1
    • +1
    /sources/view/VDateTime.cpp
12:59 pm

plucia  (deleted user)

I think VDateTime.h is less safe in this case. It should be updated to match if (_process.state() != QProcess::NotRunning) { return; } to prevent re-starting the process when its already in the "S...

I think VDateTime.h is less safe in this case. It should be updated to match 

if (_process.state() != QProcess::NotRunning) { return; }

to prevent re-starting the process when its already in the "Starting" state as well as the "Running" state.

    enum ProcessState {
        NotRunning,
        Starting,
        Running
    };
12:51 pm

plucia  (deleted user)

I've removed it now. It's the same CAN channel as in the parent class so no need to override it.

I've removed it now. It's the same CAN channel as in the parent class so no need to override it.

12:51 pm

plucia  (deleted user)

AlarmVolumeTemp has been removed

AlarmVolumeTemp has been removed

12:51 pm

plucia  (deleted user)

It is removed now

It is removed now

12:50 pm

plucia  (deleted user)

Done

Done

12:50 pm

Peter Lucia <plucia@diality.com>

DEN-8064: Addressed code review comments

Options
    • -0
    • +2
    /denali.pro
    • -0
    • +1
    /sources/canbus/MessageGlobals.h
    • -0
    • +1
    /sources/canbus/MessageInterpreter.cpp
    • -0
    • +1
    /sources/gui/GuiGlobals.h
    • -16
    • +16
    /sources/gui/qml/pages/SettingsAlarmVolume.qml
    • -2
    • +2
    /sources/gui/qml/pages/SettingsBluetooth.qml
    • -4
    • +4
    /sources/gui/qml/pages/SettingsBrightness.qml
    • -2
    • +2
    /sources/gui/qml/pages/SettingsDateTimeSet.qml
    • -2
    • +2
    /sources/gui/qml/pages/SettingsDeviceInformation.qml
    • -8
    • +7
    /sources/gui/qml/pages/SettingsHome.qml
    • -2
    • +2
    /sources/gui/qml/pages/SettingsWifi.qml
    • -0
    • +5
    /sources/model/MModel.h
    • -0
    • +45
    /sources/model/hd/adjustment/MAdjustHDAlarmVolumeResponse.cpp
    • -0
    • +82
    /sources/model/hd/adjustment/MAdjustHDAlarmVolumeResponse.h
    • -4
    • +2
    /sources/model/hd/adjustment/MTreatmentAdjustRequests.h
  16. … 3 more files in changeset.
4:34 am

Behrouz NematiPour

Updated the Alarm List

Options
    • -2
    • +4
    /sources/model/hd/alarm/MAlarmMapping.cpp
4:27 am

Behrouz NematiPour

DEN-8316: TxLog: Implementation [Simulator/Testing]

Options
    • -2
    • +5
    /denali.pro
    • -1
    • +1
    /denali.pro.user
    • -0
    • +8
    /en_US.udic
    • -0
    • +14
    /resources/license
    • binary
    /resources/settings/Instructions/1.png
    • binary
    /resources/settings/Instructions/2.png
    • binary
    /resources/settings/Instructions/3.png
    • -0
    • +8
    /resources/settings/Instructions/Instructions.conf
    • binary
    /resources/settings/Instructions/slide1_filter_v2.png
    • binary
    /resources/settings/Instructions/slide1_filter_v2b.png
    • binary
    /resources/settings/Instructions/slide2_filter-connect_v1.png
    • -0
    • +22
    /resources/settings/TreatmentParameters/DataList.conf
    • -0
    • +4
    /resources/settings/TreatmentParameters/Formats.conf
    • binary
    /resources/settings/instructions/1.png
    • binary
    /resources/settings/instructions/2.png
  16. … 39 more files in changeset.
Friday 30 Apr 2021
1:46 pm

plucia  (deleted user)

UI-DEN-7529_Service Mode
UI-DEN-7529_Service Mode
Thursday 29 Apr 2021
2:38 pm

Behrouz NematiPour

DEN-8303 Meeting & Support - General Purpose

- Adding two other Alarms parameters to the log

- Fixed the ProgressCircle component circle color.

- Adding a simple detail screen for the Treatment Log Review screen

- Popping up the keyboard when get into the Date/Time screen.

-

Options
    • -22
    • +21
    /cppcheck.err
    • -187
    • +408
    /cppcheck.log
    • -2
    • +2
    /denali.pro.user
    • -0
    • +2
    /sources/gui/GuiGlobals.h
    • -1
    • +1
    /sources/gui/qml/components/ProgressCircle.qml
    • -0
    • +4
    /sources/gui/qml/pages/posttreatment/PostTreatmentReview.qml
    • -0
    • +10
    /sources/gui/qml/pages/posttreatment/PostTreatmentStack.qml
    • -0
    • +5
    /sources/gui/qml/pages/settings/SettingsDateTime.qml
    • -1
    • +2
    /sources/gui/qml/pages/settings/SettingsStack.qml
    • -7
    • +139
    /sources/model/hd/alarm/MAlarmTriggered.cpp
    • -2
    • +28
    /sources/model/hd/alarm/MAlarmTriggered.h
11:37 am

Behrouz NematiPour

As it seems in the qml these properties are not used directly in UI and only the key/value pairs are used. So it would be better to have key or value as QStringList properties and change these prop...

As it seems in the qml these properties are not used directly in UI and only the key/value pairs are used.
So it would be better to have key or value as QStringList properties and change these properties to just values.

11:34 am

Behrouz NematiPour

For the properties, we don't use 'm' as a prefix and only start the name with no capital letter.

For the properties, we don't use 'm' as a prefix and only start the name with no capital letter.