application

Clone Tools

Stats

Commits this week: 8
Total Committers: 14
Last Commit: 23 Mar
Commits this week: 8

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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Wednesday 24 Mar 2021
2:48 pm

Peter Lucia <plucia@diality.com>

DEN-7320: Addressed code review comments

Options
    • -6
    • +7
    /scripts/date_time_set.sh
    • -0
    • +5
    /scripts/setup.sh
    • -15
    • +21
    /sources/gui/qml/globals/Variables.qml
    • -2
    • +2
    /sources/gui/qml/pages/SettingsDateTimeSet.qml
    • -2
    • +1
    /sources/gui/qml/pages/SettingsHome.qml
    • -6
    • +6
    /sources/model/dg/adjustment/MAdjustDGDateTimeResponse.h
    • -4
    • +4
    /sources/model/hd/adjustment/MAdjustHDDateTimeResponse.h
    • -6
    • +6
    /sources/model/hd/adjustment/MTreatmentAdjustRequests.h
    • -4
    • +2
    /sources/view/VDateTime.cpp
    • -2
    • +0
    /sources/view/VDateTime.h
12:46 pm

pmontazemi  (deleted user)

Are these the only measurable levels?

Are these the only measurable levels?

12:13 pm

Behrouz NematiPour

Did you do the Code Coverage? If not please remove these lines which I think are the leftovers of copy/paste. so later we test them.

Did you do the Code Coverage?
If not please remove these lines which I think are the leftovers of copy/paste.
so later we test them.

12:10 pm

Behrouz NematiPour

I see that you nicely put the _keyboard.setVisible(true) in the TextEntry component. Don't you think it would be better to put _keyboard.setVisible(false) on onEnteredPressed in the TextEntry compo...

I see that you nicely put the _keyboard.setVisible(true) in the TextEntry component.
Don't you think it would be better to put _keyboard.setVisible(false) on onEnteredPressed in the TextEntry component as well, somewehere next to the lines below?

onFocusChanged: {
    if (focus) {ome whereo
        selectAll()
        _keyboard.setVisible(true)
    }
}

onAccepted: {
    _root.enterPressed()
}
11:52 am

Behrouz NematiPour

Why onPressed? Should be onClicked.

Why onPressed?
Should be onClicked.

11:51 am

Behrouz NematiPour

Do we need to set the background color? I think the default ScreenItem backgroundColor would be fine!

Do we need to set the background color?
I think the default ScreenItem backgroundColor would be fine!

11:46 am

Behrouz NematiPour

Recently I'm removing the "Response" from the log's "infoText" function, since the log already has the source we will see it is coming from HD/DG and it make the "Response" useless and gets more sp...

Recently I'm removing the "Response" from the log's "infoText" function, since the log already has the source we will see it is coming from HD/DG and it make the "Response" useless and gets more space only.
Please apply the same for the DG.
And don't forget to update the doxygen comment as well.

11:35 am

Behrouz NematiPour

Please use a consistent "else" style. we are currently using Stroustrup style which is like: } else { or use something like: } else {

Please use a consistent "else" style.
we are currently using Stroustrup style which is like:

}
else {

or use something like:

} else {
11:22 am

Behrouz NematiPour

since you are setting the bottom margin we need to think about the object that are showing up on the bottom of the screen and since the mainMenu comes on top then it's height may not be relevant ...
  • since you are setting the bottom margin we need to think about the object that are showing up on the bottom of the screen and since the mainMenu comes on top then it's height may not be relevant to the bottom of the screen area.
    Could be better to use notificationHeight which is used for the notifications that are showing up at the bottom.
  • why it has been used by twise the size?
11:19 am

Behrouz NematiPour

Please give a better name to this rect. like _contentRect so it gives the idea of why it has been defined.

Please give a better name to this rect. like _contentRect so it gives the idea of why it has been defined.

11:09 am

Behrouz NematiPour

This process is called in the main thread (GUI thread), and any issue will freeze the UI. Let's talk about it before any decision. I need to know more about the code structure.

This process is called in the main thread (GUI thread), and any issue will freeze the UI.
Let's talk about it before any decision.
I need to know more about the code structure.

11:00 am

Behrouz NematiPour

QStringLiteral should be used for a constant string that we know is not going to change so those will be created at compile time and not every time. Here seem all the options have to run and buil...
  • QStringLiteral should be used for a constant string that we know is not going to change so those will be created at compile time and not every time.
    Here seem all the options have to run and build at runtime.
  • Why used QLatin1Char instead of only "0"?
    This is a constant leading zero.
10:37 am

Behrouz NematiPour

couldn't we use the date command instead which is more common?

couldn't we use the date command instead which is more common?

10:36 am

Behrouz NematiPour

Please put more description of why we should have these two lines?

Please put more description of why we should have these two lines?

10:32 am

Behrouz NematiPour

Wouldn't be better to define just one set of common "settings..." variables and bind the others to that instead of having a chain of binding? Preffered: settings<1>: 85 settings<2>: 75 settingsBLE...

Wouldn't be better to define just one set of common "settings..." variables and bind the others to that instead of having a chain of binding?
Preffered:

settings<1>: 85
settings<2>: 75

settingsBLE<1>:settings<1>
....
settingsWifi<1>: settings<1>
...

instead of the chain of:

settings<1>: 85
settings<2>: 75

settingsBLE<1>:settings<1>
....
settingsWifi<1>: settingsBLE<1>
...
Tuesday 23 Mar 2021
11:06 pm

Behrouz NematiPour

DEN-7461: Meetings & Support - Pre-Treatment Integration

- Integration modifications

- UF Modifications request

- Saline Modification request

- Updated the validation mAccepted to be 1=true and 0=false.

- Heparin Stopped stated modification request

Options
    • -2
    • +2
    /denali.pro.user
    • -1
    • +2
    /sources/gui/qml/components/Footer.qml
    • -1
    • +1
    /sources/gui/qml/components/TextRect.qml
    • -1
    • +2
    /sources/gui/qml/compounds/InstructionView.qml
    • -2
    • +2
    /sources/gui/qml/compounds/StepNavigationTitleBar.qml
    • -3
    • +6
    /sources/gui/qml/pages/MainStack.qml
    • -0
    • +3
    /sources/gui/qml/pages/ManagerStack.qml
    • -0
    • +3
    /sources/gui/qml/pages/SettingsStack.qml
    • -1
    • +3
    /sources/gui/qml/pages/pretreatment/PreTreatmentBase.qml
    • -5
    • +9
    /sources/gui/qml/pages/pretreatment/PreTreatmentConsumablesStack.qml
    • -2
    • +7
    /sources/gui/qml/pages/pretreatment/PreTreatmentDisposablesStack.qml
    • -0
    • +3
    /sources/gui/qml/pages/pretreatment/PreTreatmentMainStack.qml
    • -26
    • +48
    /sources/gui/qml/pages/pretreatment/PreTreatmentPrimeStack.qml
    • -0
    • +3
    /sources/gui/qml/pages/pretreatment/PreTreatmentWaterSampleStack.qml
    • -0
    • +3
    /sources/gui/qml/pages/pretreatment/connection/PreTreatmentConnectionStack.qml
  16. … 17 more files in changeset.
2:21 pm

Peter Lucia <plucia@diality.com>

DEN-7523, DEN-7050: Upon bootup, connection to DialityWEP, DialityWAPA, DialityWAPB, and DilaityIoT succeeded with ping test.

Options
    • -1
    • +1
    /denali.pro.user
    • -9
    • +9
    /scripts/run.sh
    • -0
    • +1
    /scripts/wifi_read_gateway.sh
    • -0
    • +1
    /scripts/wifi_request_auto_assigned_ip.sh
    • -1
    • +3
    /scripts/wifi_start_wpa_supplicant.sh
    • -2
    • +8
    /sources/gui/qml/components/TextEntry.qml
    • -0
    • +18
    /sources/gui/qml/pages/SettingsWifi.qml
    • -0
    • +2
    /sources/view/VNetworkModel.cpp
    • -0
    • +1
    /sources/view/VNetworkModel.h
Monday 22 Mar 2021
10:41 pm

Behrouz NematiPour

DEN-7408: preTx: Implementation [Navigation] ( 3 )

- Added Rejection reason texts.

- Removed unused view.

- Fixed heparin ranges.

Options
    • -2
    • +0
    /denali.pro
    • -1
    • +1
    /denali.pro.user
    • -1
    • +0
    /sources/gui/GuiGlobals.cpp
    • -0
    • +3
    /sources/gui/qml/components/NotificationBar.qml
    • -2
    • +0
    /sources/gui/qml/main.qml
    • -4
    • +24
    /sources/gui/qml/pages/pretreatment/PreTreatmentPrimeStack.qml
    • -0
    • +10
    /sources/gui/qml/pages/pretreatment/PreTreatmentWaterSampleStack.qml
    • -2
    • +22
    /sources/gui/qml/pages/pretreatment/connection/PreTreatmentConnectionStack.qml
    • -15
    • +0
    /sources/view/VTreatmentBegin.cpp
    • -44
    • +0
    /sources/view/VTreatmentBegin.h
    • -4
    • +4
    /sources/view/VTreatmentCreate.h
    • -1
    • +0
    /sources/view/VView.h
    • -10
    • +0
    /unittests/tst_views.cpp
    • -2
    • +0
    /unittests/tst_views.h
7:28 pm

Behrouz NematiPour

DEN-7518: preTx: Implementation [Screens] ( 3 )

- The first round of the pre-treatment flow works.

Options
    • -14
    • +34
    /denali.pro
    • -10
    • +10
    /sources/canbus/MessageDispatcher.cpp
    • -1
    • +1
    /sources/canbus/MessageDispatcher.h
    • -3
    • +3
    /sources/canbus/MessageGlobals.h
    • -9
    • +9
    /sources/canbus/MessageInterpreter.cpp
    • -2
    • +10
    /sources/gui/GuiGlobals.cpp
    • -3
    • +3
    /sources/gui/GuiGlobals.h
    • -12
    • +33
    /sources/gui/qml/main.qml
    • -1
    • +1
    /sources/gui/qml/pages/MainStack.qml
    • -3
    • +6
    /sources/gui/qml/pages/pretreatment/PreTreatmentConsumablesStack.qml
    • -3
    • +1
    /sources/gui/qml/pages/pretreatment/PreTreatmentDisposablesStack.qml
    • -7
    • +7
    /sources/gui/qml/pages/pretreatment/PreTreatmentPrimeStack.qml
    • -2
    • +5
    /sources/gui/qml/pages/pretreatment/PreTreatmentWaterSampleStack.qml
    • -11
    • +10
    /sources/gui/qml/pages/pretreatment/connection/PreTreatmentConnectionStack.qml
    • -8
    • +0
    /sources/gui/qml/pages/pretreatment/create/PreTreatmentCreate.qml
  16. … 52 more files in changeset.
11:20 am

pmontazemi  (deleted user)

No Msg Box?

No Msg Box?

11:20 am

pmontazemi  (deleted user)

No Msg Box?

No Msg Box?

10:48 am

Behrouz NematiPour

DEN-7420: preTx: Implementation [Messaging] ( 3 )

Options
    • -17
    • +32
    /denali.pro
    • -1
    • +1
    /denali.pro.user
    • -0
    • +2
    /en_US.udic
    • -41
    • +144
    /sources/canbus/MessageDispatcher.cpp
    • -6
    • +17
    /sources/canbus/MessageDispatcher.h
    • -19
    • +40
    /sources/canbus/MessageGlobals.h
    • -39
    • +52
    /sources/canbus/MessageInterpreter.cpp
    • -1
    • +1
    /sources/gui/GuiGlobals.cpp
    • -16
    • +38
    /sources/gui/GuiGlobals.h
    • -2
    • +2
    /sources/gui/qml/main.qml
    • -3
    • +3
    /sources/gui/qml/pages/pretreatment/PreTreatmentDisposablesStack.qml
    • -11
    • +11
    /sources/gui/qml/pages/pretreatment/PreTreatmentPrimeStack.qml
    • -48
    • +136
    /sources/model/MModel.h
    • -41
    • +0
    /sources/model/MPowerOff.cpp
    • -72
    • +0
    /sources/model/MPowerOff.h
  16. … 50 more files in changeset.
Thursday 18 Mar 2021
12:55 pm

Behrouz NematiPour

DEN-7518: preTx: Implementation [Screens] ( 3 )

Options
    • -1
    • +1
    /denali.pro.user
    • -3
    • +4
    /denali.qrc
    • -3
    • +4
    /sources/gui/qml/compounds/InstructionView.qml
    • -2
    • +5
    /sources/gui/qml/globals/Variables.qml
    • -2
    • +0
    /sources/gui/qml/pages/pretreatment/PreTreatmentConsumablesStack.qml
    • -23
    • +4
    /sources/gui/qml/pages/pretreatment/PreTreatmentMainStack.qml
    • -0
    • +121
    /sources/gui/qml/pages/pretreatment/PreTreatmentPrimeStack.qml
    • -0
    • +103
    /sources/gui/qml/pages/pretreatment/connection/PreTreatmentConnectionStack.qml
    • -0
    • +85
    /sources/gui/qml/pages/pretreatment/connection/PreTreatmentUltrafiltration.qml
    • -1
    • +1
    /sources/gui/qml/pages/pretreatment/create/PreTreatmentCreateStack.qml
    • -122
    • +0
    /sources/gui/qml/pages/pretreatment/prime/PreTreatmentPrimeStack.qml
    • -126
    • +0
    /sources/gui/qml/pages/pretreatment/prime/PreTreatmentUltrafiltration.qml
11:06 am

Peter Lucia <plucia@diality.com>

DEN-7050, DEN-7522, DEN-7523: Improved auto dhcp assignment time, connected to DialityWEP, DialityWAPA, DialityWAPB, and DilaityIoT consecutively in one test.

Options
    • -366
    • +366
    /denali.pro.user
    • -1
    • +1
    /scripts/copy.sh
    • -0
    • +10
    /scripts/run.sh
    • -0
    • +7
    /scripts/wifi_disconnect_network.sh
    • -0
    • +7
    /scripts/wifi_generate_wpa_supplicant.sh
    • -0
    • +5
    /scripts/wifi_read_dns.sh
    • -0
    • +6
    /scripts/wifi_read_gateway.sh
    • -0
    • +6
    /scripts/wifi_read_ip_settings.sh
    • -34
    • +6
    /scripts/wifi_request_auto_assigned_ip.sh
    • -1
    • +7
    /scripts/wifi_reset_adapter.sh
    • -3
    • +9
    /scripts/wifi_reset_interface.sh
    • -0
    • +12
    /scripts/wifi_save_dhcp_interface.sh
    • -0
    • +9
    /scripts/wifi_save_static_interface_assignments.sh
    • -1
    • +7
    /scripts/wifi_scan_for_networks.sh
    • -0
    • +8
    /scripts/wifi_set_auto_assigned_ip.sh
  16. … 6 more files in changeset.
1:50 am

Behrouz NematiPour

DEN-7139: preTx: Implementation [Screens] ( 2 )

Options
    • -1
    • +1
    /denali.pro.user
    • -25
    • +6
    /denali.qrc
    • -1
    • +1
    /sources/gui/qml/components/Circle.qml
    • -0
    • +35
    /sources/gui/qml/components/Footer.qml
    • -0
    • +1
    /sources/gui/qml/components/ProgressCircle.qml
    • -1
    • +1
    /sources/gui/qml/components/StepIndicator.qml
    • -0
    • +68
    /sources/gui/qml/components/TimeCircle.qml
    • -5
    • +5
    /sources/gui/qml/compounds/CheckListView.qml
    • -20
    • +9
    /sources/gui/qml/compounds/InstructionView.qml
    • -79
    • +0
    /sources/gui/qml/compounds/PreTreatmentTitleBar.qml
    • -0
    • +68
    /sources/gui/qml/compounds/StepNavigationTitleBar.qml
    • -1
    • +2
    /sources/gui/qml/globals/Fonts.qml
    • -1
    • +8
    /sources/gui/qml/globals/Variables.qml
    • -2
    • +2
    /sources/gui/qml/pages/MainStack.qml
    • -10
    • +107
    /sources/gui/qml/pages/pretreatment/PreTreatmentBase.qml
  16. … 25 more files in changeset.
Wednesday 17 Mar 2021
12:36 pm

Peter Lucia <plucia@diality.com>

DEN-7050: Added missing echo to dns script

Options
    • -1
    • +1
    /scripts/wifi_set_dns.sh
12:32 pm

Peter Lucia <plucia@diality.com>

DEN-7050: Added missing " in dns script

Options
    • -1
    • +1
    /scripts/wifi_set_dns.sh
12:27 pm

Peter Lucia <plucia@diality.com>

DEN-7050: Updated set dns script

Options
    • -4
    • +5
    /scripts/wifi_set_dns.sh
10:19 am

Peter Lucia <plucia@diality.com>

DEN-7315: Removed unneeded import

Options
    • -1
    • +0
    /sources/view/VAlert.h
9:24 am

plucia  (deleted user)

UI-DEN-7035_Date Time Set
UI-DEN-7035_Date Time Set