application

Clone Tools

Stats

Commits this week: 0
Total Committers: 14
Last Commit: 16 Dec
Commits this week: 0

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
Wednesday 15 Feb 2023
1:14 pm

Vy <vduong@sunriselabs.com>

DEN-14743 Modify to only require single time pairing for data sync

Options
    • -26
    • +106
    /sources/bluetooth/BluetoothInterface.cpp
    • -3
    • +16
    /sources/bluetooth/BluetoothInterface.h
    • -0
    • +1
    /sources/view/settings/VBluetooth.cpp
Friday 10 Feb 2023
10:12 am

Dara Navaei

Bamboo Commit: Updated MAlarmMapping.cpp

Options
    • -14
    • +20
    /sources/model/hd/alarm/MAlarmMapping.cpp
10:12 am

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -1
    • +1
    /cppcheck.sh
    • -1
    • +1
    /imake.sh
    • -1
    • +1
    /main.cpp
    • -1
    • +1
    /sources/canbus/CanInterface.h
    • -1
    • +1
    /sources/canbus/FrameInterface.h
    • -1
    • +1
    /sources/canbus/MessageAcknowModel.h
    • -1
    • +1
    /sources/canbus/MessageGlobals.h
    • -1
    • +1
    /sources/canbus/MessageInterpreter.cpp
    • -1
    • +1
    /sources/cloudsync/CloudSyncController.cpp
    • -1
    • +1
    /sources/cloudsync/CloudSyncController.h
    • -1
    • +1
    /sources/device/DeviceController.cpp
    • -1
    • +1
    /sources/gui/GuiController.cpp
    • -1
    • +1
    /sources/gui/GuiView.h
    • -1
    • +1
    /sources/gui/qml/PowerItem.qml
    • -1
    • +1
    /sources/gui/qml/components/DebugDataColumn.qml
  16. … 36 more files in changeset.
9:51 am

Behrouz NematiPour

Merge branch 'DEN-14733-UI-BN-S90-2' into staging

Options
    • -2
    • +3
    /AlarmMapping.csv
    • -39
    • +1221
    /cppcheck.err
    • -69
    • +271
    /cppcheck.log
    • -118
    • +66
    /denali.pro.user
    • -0
    • +1
    /en_US.udic
    • -6
    • +0
    /imake.sh
    • -0
    • +31
    /main.cpp
    • -1
    • +1
    /sources/canbus/CanInterface.h
    • -4
    • +4
    /sources/canbus/MessageGlobals.h
    • -5
    • +5
    /sources/canbus/MessageInterpreter.cpp
    • -1
    • +6
    /sources/device/DeviceController.cpp
    • -11
    • +2
    /sources/gui/GuiController.cpp
    • -0
    • +3
    /sources/gui/GuiView.h
    • -4
    • +1
    /sources/gui/qml/PowerItem.qml
    • -3
    • +3
    /sources/gui/qml/components/DebugDataColumn.qml
  16. … 29 more files in changeset.
9:30 am

Behrouz NematiPour

Merge branch 'DEN-14517-UI-BN-S88' into staging

Options
    • -205
    • +205
    /cppcheck.log
    • -18
    • +70
    /denali.pro.user
    • -1
    • +7
    /imake.sh
9:23 am

Behrouz NematiPour

9:23 am

Behrouz NematiPour

9:23 am

Michael Garthwaite , vduong  (deleted user) and jishii  (deleted user)

9:20 am

Sean Nash and vduong  (deleted user)

9:18 am

vduong  (deleted user)

Yes, assignments break bindings. RESOLVED

Yes, assignments break bindings.

RESOLVED

9:17 am

vduong  (deleted user)

Ok, sounds good. RESOLVED

Ok, sounds good.

RESOLVED

8:23 am

Behrouz NematiPour

Good point, But you can't. The readonly properties are only working with bindings, not assignments. I am using that as a public variable.

Good point,
But you can't.
The readonly properties are only working with bindings, not assignments.
I am using that as a public variable.

8:17 am

Behrouz NematiPour

Good point, I agree with the two intervals, it would be clarifying. For the counter, though, it might not be always the case. I will start doing it in my next task, since I would like to change all...

Good point,
I agree with the two intervals, it would be clarifying.
For the counter, though, it might not be always the case.
I will start doing it in my next task, since I would like to change all, to be consistent across the entire code base.

8:14 am

vduong  (deleted user)

If _root.isOpen is not modified outside of this powerItem.qml, You can make it a readonly property and bind it to the powerDialog's property.

If _root.isOpen is not modified outside of this powerItem.qml, You can make it a readonly property and bind it to the powerDialog's property.

7:40 am

vduong  (deleted user)

Only a suggestion: It is a good idea to append the unit to the end of the variable name especially for time. It helps improve readability and clarity. For instance, here, we have ms and the next o...

Only a suggestion:
It is a good idea to append the unit to the end of the variable name especially for time. It helps improve readability and clarity.

For instance, here, we have ms and the next one, _pendingInterval is in seconds. and I believe _pendingCounter is in minutes based on the .cpp file's comment.

_interval_ms
_pendingInterval_sec
_pendingCounter_min

7:16 am

vduong  (deleted user)

RESOLVED

RESOLVED

7:12 am

Behrouz NematiPour

Thanks for the feedback. Fixed.

Thanks for the feedback.
Fixed.

7:12 am

Behrouz NematiPour

CR comment: UI-DEN-14517-1#c16470

http://devapps.diality.us:8060/cru/UI-DEN-14517-1#c16470

Options
    • -1
    • +1
    /sources/cloudsync/CloudSyncController.cpp
7:03 am

vduong  (deleted user)

The variable, destination, itself is a QString, you don't need to do QString("%1").arg(destination), you can pass destination in directly.

The variable, destination, itself is a QString, you don't need to do QString("%1").arg(destination), you can pass destination in directly.

6:41 am

Behrouz NematiPour

DEN-14736 : IMP: DEN-14768 : InTx UF Adj

DEN-14768

Ultrafiltration Adjustment Screen Slider Does Not Update Maximum UF Vol after Duration Settings Adjustment

Options
    • -10
    • +12
    /sources/gui/qml/components/Slider.qml
Thursday 09 Feb 2023
10:40 pm

Behrouz NematiPour

DEN-14736 : IMP: Updated MsgID 0x2D00

/*!

* \brief The MDGTemperatures class

* \details The DG temperatures data model

*

* | MSG | CAN ID | Box | Type | Ack | Src | Dst | Description |

* |:----:|:------:|:---:|:------:|:---:|:---:|:---:|:-----------: |

* |0x2D00| 0x080 | 8 | 2 Hz | N | DG | All | DG Temperatures Data |

*

* | Payload ||

* | ||

* | #01:(F32) | \ref Data::mInletPrimaryHeater |

* | #02:(F32) | \ref Data::mHeatDisinfect |

* | #03:(F32) | \ref Data::mOutletPrimaryHeater |

* | #04:(F32) | \ref Data::mConductivitySensor1 |

* | #05:(F32) | \ref Data::mConductivitySensor2 |

* | #06:(F32) | \ref Data::mOutletRedundancy |

* | #07:(F32) | \ref Data::mInletDialysate |

* | #08:(F32) | \ref Data::mPrimaryHeaterThermoCouple |

* | #09:(F32) | \ref Data::mTrimmerHeaterThermoCouple |

* | #10:(F32) | \ref Data::mPrimaryHeaterColdJunction |

* | #11:(F32) | \ref Data::mTrimmerHeaterColdJunction |

* | #12:(F32) | \ref Data::mPrimaryHeaterInternal |

* | #13:(F32) | \ref Data::mTrimmerHeaterInternal |

* | #14:(F32) | \ref Data::mFpgaBoard |

* | #15:(F32) | \ref Data::mLoadCellA1B1 |

* | #16:(F32) | \ref Data::mLoadCellA2B2 |

* | #17:(F32) | \ref Data::mInternalTHDORTD |

* | #18:(F32) | \ref Data::mInternalTDIRTD |

* | #19:(F32) | \ref Data::mInteralTHDRTD |

* | #20:(F32) | \ref Data::mInternalCondSnsrTemp |

* | #21:(F32) | \ref Data::mBaroTempSensor |

*

* \sa Data

*

* <h2 class="groupheader">Logging info</h2>

* | ||

* | ||

* | typeText | Datum |

* | unitText | DG |

* | infoText | Temperatures |

*

*/

Options
    • -2
    • +2
    /sources/canbus/MessageGlobals.h
    • -3
    • +3
    /sources/gui/qml/components/DebugDataColumn.qml
    • -7
    • +23
    /sources/gui/qml/dialogs/DiagnosticsDialog.qml
    • -13
    • +45
    /sources/model/dg/data/MDGTemperaturesData.cpp
    • -26
    • +50
    /sources/model/dg/data/MDGTemperaturesData.h
    • -13
    • +21
    /sources/view/dg/data/VDGTemperaturesData.cpp
    • -0
    • +8
    /sources/view/dg/data/VDGTemperaturesData.h
8:56 pm

Behrouz NematiPour

DEN-14736 : IMP: Updated MsgID 0x2000

/*!

* \brief The MDGPressures class

* \details The DG pressures data model

*

* | MSG | CAN ID | Box | Type | Ack | Src | Dst | Description |

* |:----:|:------:|:---:|:------:|:---:|:---:|:---:|:-----------: |

* |0x2000| 0x080 | 8 | 1 Hz | N | DG | All | DG Pressures Data |

*

* | Payload ||

* | ||

* | #1:(F32) | \ref Data::mROInletPSI |

* | #2:(F32) | \ref Data::mROOutletPSI |

* | #3:(F32) | \ref Data::mDrainInletPSI |

* | #4:(F32) | \ref Data::mDrainOutletPSI |

* | #5:(F32) | \ref Data::mBarometricPSI |

*

* \sa Data

*

* <h2 class="groupheader">Logging info</h2>

* | ||

* | ||

* | typeText | Datum |

* | unitText | DG |

* | infoText | Pressures |

*

*/

Options
    • -10
    • +15
    /sources/gui/qml/dialogs/DiagnosticsDialog.qml
    • -12
    • +16
    /sources/model/dg/data/MDGPressuresData.cpp
    • -4
    • +7
    /sources/model/dg/data/MDGPressuresData.h
    • -4
    • +5
    /sources/view/dg/data/VDGPressuresData.cpp
    • -4
    • +5
    /sources/view/dg/data/VDGPressuresData.h
8:55 pm

Behrouz NematiPour

DEN-14736 : IMP: Updated MsgID 0x5C00

/*!

* \brief The MTreatmentStates class

* \details The treatment states data model

* These are states of the current treatment.

* It also includes the ultrafiltration states, saline bolus states.

*

* | MSG | CAN ID | Box | Type | Ack | Src | Dst | Description |

* |:----:|:------:|:---:|:------:|:---:|:---:|:---:|:-----------: |

* |0x5C00| 0x040 | 7 | 1 Hz | N | HD | All | Pre-Treatment States Data |

*

* | Payload ||

* | ||

* | #1:(U32) | \ref Data::mSubMode |

* | #2:(U32) | \ref Data::mWaterSampleState |

* | #3:(U32) | \ref Data::mSelfTestConsumablesState |

* | #4:(U32) | \ref Data::mSelfTestNoCartridgeState |

* | #5:(U32) | \ref Data::mInstallationState |

* | #6:(U32) | \ref Data::mSelfTestDryState |

* | #7:(U32) | \ref Data::mPrimeState |

* | #8:(U32) | \ref Data::mRecirculateState |

* | #9:(U32) | \ref Data::mPatientConnectionState |

* | #9:(U32) | \ref Data::mWetSelfTestsState |

* | #9:(U32) | \ref Data::mPreTreatmentRsrvrState |

*

* \sa Data

* \sa HD_Pre_Treatment_Mode_States

* \sa HD_Pre_Treatment_Sample_Water_States

* \sa HD_Pre_Treatment_Consumable_Self_Test_States

* \sa HD_Pre_Treatment_No_Cart_Self_Tests_States

* \sa HD_Pre_Treatment_Dry_Self_Tests_States

* \sa HD_Pre_Treatment_Prime_States

* \sa HD_Pre_Treatment_Recirc_States

*

* <h2 class="groupheader">Logging info</h2>

* | ||

* | ||

* | typeText | Datum |

* | unitText | HD |

* | infoText | PreTreatmentStates |

*

*/

Options
    • -1
    • +1
    /sources/canbus/MessageGlobals.h
    • -0
    • +8
    /sources/model/hd/data/pretreatment/MPreTreatmentStatesData.cpp
    • -29
    • +35
    /sources/model/hd/data/pretreatment/MPreTreatmentStatesData.h
    • -0
    • +2
    /sources/view/hd/data/pretreatment/VPreTreatmentStatesData.cpp
    • -0
    • +2
    /sources/view/hd/data/pretreatment/VPreTreatmentStatesData.h
7:19 pm

Behrouz NematiPour

DEN-14736 : IMP: Updated MsgID 0x2C00

/*!

* \brief The MDGHeaters class

* \details The DG heaters data model

*

* | MSG | CAN ID | Box | Type | Ack | Src | Dst | Description |

* |:----:|:------:|:---:|:------:|:---:|:---:|:---:|:-----------: |

* |0x2C00| 0x080 | 8 | 2 Hz | N | DG | All | DG Heaters Data |

*

* | Payload ||

* | ||

* | # 1:(F32) | \ref Data::mMainPrimayHeaterDC |

* | # 2:(F32) | \ref Data::mSmallPrimaryHeaterDC |

* | # 3:(F32) | \ref Data::mTrimmerHeaterDC |

* | # 4:(F32) | \ref Data::mPrimaryTargetTemp |

* | # 5:(F32) | \ref Data::mTrimmerTargetTemp |

* | # 6:(F32) | \ref Data::mPrimaryHeaterState |

* | # 7:(F32) | \ref Data::mTrimmerHeaterState |

* | # 8:(F32) | \ref Data::mPrimaryEfficiency |

* | # 9:(F32) | \ref Data::mPrimaryCalcTargetTemp |

* | #10:(F32) | \ref Data::mTrimmerCalcCurrentTemp |

* | #11:(F32) | \ref Data::mTrimmerUseLastDC |

*

* \sa Data

*

* <h2 class="groupheader">Logging info</h2>

* | ||

* | ||

* | typeText | Datum |

* | unitText | DG |

* | infoText | Heaters |

*

*/

Options
    • -1
    • +1
    /sources/canbus/MessageGlobals.h
    • -3
    • +3
    /sources/gui/qml/dialogs/DiagnosticsDialog.qml
    • -9
    • +42
    /sources/model/dg/data/MDGHeatersData.cpp
    • -10
    • +33
    /sources/model/dg/data/MDGHeatersData.h
    • -3
    • +11
    /sources/view/dg/data/VDGHeatersData.cpp
    • -4
    • +12
    /sources/view/dg/data/VDGHeatersData.h
    • -16
    • +16
    /unittests/tst_models.cpp
4:39 pm

jishii  (deleted user)

4:38 pm

Behrouz NematiPour

DEN-14736 : IMP: DEN-14486

- DEN-14486 : Incorrect alarm notification appears during DG Flush

Options
    • -1
    • +1
    /cppcheck.err
    • -15
    • +15
    /cppcheck.log
    • -1
    • +1
    /sources/model/hd/alarm/MAlarmMapping.cpp
    • -3
    • +32
    /sources/view/hd/alarm/VAlarmActiveList.cpp
    • -1
    • +6
    /sources/view/hd/alarm/VAlarmActiveList.h
    • -0
    • +6
    /sources/view/hd/alarm/VAlarmStatus.cpp
    • -1
    • +1
    /sources/view/hd/alarm/VAlarmStatus.h
1:55 pm

Vy <vduong@sunriselabs.com>

DEN-14743 Bluetooth bug fixes - improve connection handling and message display

Options
    • -10
    • +101
    /sources/bluetooth/BluetoothInterface.cpp
    • -1
    • +2
    /sources/bluetooth/BluetoothInterface.h
    • -1
    • +1
    /sources/gui/qml/pages/settings/SettingsBluetoothCuff.qml
    • -0
    • +1
    /sources/model/settings/MBluetooth.h
    • -1
    • +5
    /sources/view/settings/VBluetooth.cpp
11:08 am

Sean Nash

So we will likely remove this TODO and comment later? RESOLVED.

So we will likely remove this TODO and comment later?
RESOLVED.

11:06 am

Sean Nash

RESOLVED.

RESOLVED.

10:07 am

Behrouz NematiPour

It was removed. UI sends an alarmTriggered to HD for that, and should not display a dialog by itself. Since on the device, if there is no other alarm HD will eventually display it, and if there is ...

It was removed.
UI sends an alarmTriggered to HD for that, and should not display a dialog by itself.
Since on the device, if there is no other alarm HD will eventually display it, and if there is HD will override it.
It was just a bug in development then, which was always displaying a useless dialog on VM that doesn't even have a physical SD Card.