dd-firmware / firmware

App

Clone Tools

Stats

Commits this week: 35
Total Committers: 16
Last Commit: 11:49
Commits this week: 35

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: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
    ./Controllers/BalancingChamber.c
    • -4
    • +18
    ./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
    ./DDCommon.h
9:11 am

Raghu Kallala

resolved merge conflicts for LDT-3677

Options
    • -1
    • +1
    ./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
    ./Modes/FPModes/FPOperationModes.c
8:33 am

Raghu Kallala

LDT-3749 addressed review comments

Options
    • -1
    • +3
    ./Modes/ModeFault.c
    • -1
    • +0
    ./Modes/ModeGenDialysate.c
    • -1
    • +3
    ./Modes/ModePostGenDialysate.c
    • -1
    • +3
    ./Modes/ModePreGenDialysate.c
    • -1
    • +3
    ./Modes/ModeStandby.c
    • -0
    • +1
    ./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
    ./Drivers/ConductivitySensors.c
    • -1
    • +0
    ./Monitors/Conductivity.c
Wednesday 08 Apr
5:16 pm

Vendor - TEL - Arpita Srivastava

LDT-3735: Fixed dependency between NVM files.

Options
    • -1
    • +12
    ./Drivers/NVDriver.c
    • -0
    • +19
    ./Drivers/NVDriver.h
    • -11
    • +27
    ./Services/NVMgmtDD.c
    • -5
    • +11
    ./Services/NVMgmtDD.h
    • -10
    • +22
    ./Services/NVMsgQ.c
    • -1
    • +15
    ./Services/NVMsgQ.h
    • -5
    • +21
    ./Services/NVRecordsDD.c
    • -90
    • +93
    ./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?

1:44 pm

Sean Nash

Vinayakam Mani, how does that signal happen normally? In your example, when would DD signal FP when starting pre-gen mode? If that signal happened in pre-gen transition function, it would always ha...

Vinayakam Mani, how does that signal happen normally? In your example, when would DD signal FP when starting pre-gen mode? If that signal happened in pre-gen transition function, it would always happen regardless of how we got to pre-gen mode.

1:41 pm

Sean Nash

Add comment that these modes not (yet?) supported.

Add comment that these modes not (yet?) supported.

1:32 pm

Vendor - TEL - Jashwant Gantyada

LDT-3103 addressed code review comments and merged latest staging

Options
    • -2
    • +2
    ./Controllers/BalancingChamber.c
1:30 pm

Vendor - TEL - Jashwant Gantyada

Merge branch 'staging' into LDT-3103-closed-loop-control-pressure-threshold-fix

Options
    • -2
    • +24
    ./Controllers/BalancingChamber.c
    • -4
    • +18
    ./Modes/ModeGenDialysate.c
12:25 pm

Sean Nash

Why do some functions take sensor as a U32 and some as a CONDUCTIVITY_SENSORS_T? The latter seems appropriate.

Why do some functions take sensor as a U32 and some as a CONDUCTIVITY_SENSORS_T? The latter seems appropriate.

12:24 pm

Sean Nash

Remove blank line.

Remove blank line.

12:24 pm

Sean Nash

Remove blank line.

Remove blank line.

12:23 pm

Sean Nash

Add blank line before return.

Add blank line before return.

12:22 pm

Sean Nash

Add blank line before return.

Add blank line before return.

12:19 pm

Sean Nash

Remove blank line.

Remove blank line.

12:11 pm

Sean Nash

Why are we switching sensors around?

Why are we switching sensors around?