dd-firmware

Clone Tools

Stats

Commits this week: 27
Total Committers: 16
Last Commit: 08:40
Commits this week: 27

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
Friday 10 Apr
10:36 am

Raghu Kallala

LDT-3677 merged into staging

Options
10:25 am

Vendor - TEL - Sameer Poyil

its auto generated . should not change it manually. any way when merging its added correct value

its auto generated . should not change it manually. any way when merging its added correct value

10:24 am

Vendor - TEL - Sameer Poyil

Now we don't need this macro ?

Now we don't need this macro ?

10:17 am

Vendor - TEL - Sameer Poyil

now we are initializing sensor simultaneously?

now we are initializing sensor simultaneously?

10:16 am

Vendor - TEL - Sameer Poyil

comments missing

comments missing

10:15 am

Vendor - TEL - Sameer Poyil

comments missing

comments missing

10:15 am

Vendor - TEL - Sameer Poyil

This is not clear- please give a detailed description for this variable

This is not clear- please give a detailed description for this variable

10:14 am

Vendor - TEL - Sameer Poyil

use capital for starting letter

use capital for starting letter

10:13 am

Vendor - TEL - Sameer Poyil

please align the comments

please align the comments

10:13 am

Vendor - TEL - Sameer Poyil

please Alignment the comments

please Alignment the comments

10:10 am

Vendor - TEL - Sameer Poyil

Why the order is important here ? please document it

Why the order is important here ? please document it

10:08 am

Vendor - TEL - Arpita Srivastava

Merge branch 'staging' of bitbucket.org:diality-cloud/dd-firmware into LDT-1195-dd-is-balancing-chamber-switching

Options
    • -2
    • +24
    /firmware/App/Controllers/BalancingChamber.c
    • -4
    • +18
    /firmware/App/Modes/ModeGenDialysate.c
10:06 am

Vendor - TEL - Sameer Poyil

10:06 am

Vendor - TEL - Sameer Poyil

do we need bracket here ? all text should start with capital

do we need bracket here ? all text should start with capital

10:02 am

Sean Nash and Michael Garthwaite

10:00 am

Vendor - TEL - Sameer Poyil

Why a * here ?

Why a * here ?

9:43 am

Vendor - TEL - Sameer Poyil

is it write states or read state ?

is it write states or read state ?

9:43 am

Vendor - TEL - Sameer Poyil

is documentation correct ?

is documentation correct ?

9:12 am

Dara Navaei

Bamboo Commit: Updated DDCommon.h with build versions from Bamboo.

Options
    • -1
    • +1
    /firmware/App/DDCommon.h
9:11 am

Raghu Kallala

resolved merge conflicts for LDT-3677

Options
    • -1
    • +1
    /firmware/App/Services/Messaging.c
9:02 am

Michael Garthwaite

8:57 am

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3677_Request: add SW config to enable defeatured IOFP control in the IOFP FW
LEAHI-DD-FIRMWARE-LDT-3677_Request: add SW config to enable defeatured IOFP control in the IOFP FW
8:46 am

Raghu Kallala

LDT-3677 added fix to reset value when TC is disabled for Boost pump and defeatured

Options
    • -0
    • +10
    /firmware/App/Modes/FPModes/FPOperationModes.c
8:33 am

Raghu Kallala

LDT-3749 addressed review comments

Options
    • -1
    • +3
    /firmware/App/Modes/ModeFault.c
    • -1
    • +0
    /firmware/App/Modes/ModeGenDialysate.c
    • -1
    • +3
    /firmware/App/Modes/ModePostGenDialysate.c
    • -1
    • +3
    /firmware/App/Modes/ModePreGenDialysate.c
    • -1
    • +3
    /firmware/App/Modes/ModeStandby.c
    • -0
    • +1
    /firmware/App/Modes/OperationModes.c
8:21 am

Raghu Kallala

You are right, modified in all places

You are right, modified in all places

Thursday 09 Apr
9:04 am

Michael Garthwaite

LDT-2008: code review comments.

Options
    • -6
    • +8
    /firmware/App/Drivers/ConductivitySensors.c
    • -1
    • +0
    /firmware/App/Monitors/Conductivity.c
Wednesday 08 Apr
5:16 pm

Vendor - TEL - Arpita Srivastava

LDT-3735: Fixed dependency between NVM files.

Options
    • -1
    • +12
    /firmware/App/Drivers/NVDriver.c
    • -0
    • +19
    /firmware/App/Drivers/NVDriver.h
    • -11
    • +27
    /firmware/App/Services/NVMgmtDD.c
    • -5
    • +11
    /firmware/App/Services/NVMgmtDD.h
    • -10
    • +22
    /firmware/App/Services/NVMsgQ.c
    • -1
    • +15
    /firmware/App/Services/NVMsgQ.h
    • -5
    • +21
    /firmware/App/Services/NVRecordsDD.c
    • -90
    • +93
    /firmware/App/Services/NVRecordsDD.h
1:51 pm

Sean Nash

FALSE?

FALSE?

1:50 pm

Sean Nash

FALSE?

FALSE?

1:46 pm

Sean Nash

Should default be TRUE? If we are being asked to validate an jump to an invalid state, shouldn't we return FALSE?

Should default be TRUE? If we are being asked to validate an jump to an invalid state, shouldn't we return FALSE?