dd-firmware / firmware

App

Clone Tools

Stats

Commits this week: 48
Total Committers: 15
Last Commit: 14:29
Commits this week: 48

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
Today
10 hrs ago

Raghu Kallala

LDT-3603 updated comment wording

Options
    • -1
    • +1
    ./Modes/FPModes/FPOperationModes.c
10 hrs ago

Raghu Kallala

LDT-3603 initial draft for FP set operation sub mode

Options
    • -0
    • +88
    ./Modes/FPModes/FPOperationModes.c
    • -0
    • +2
    ./Modes/FPModes/FPOperationModes.h
    • -0
    • +35
    ./Modes/FPModes/ModeGenPermeate.c
    • -1
    • +1
    ./Modes/FPModes/ModeGenPermeate.h
    • -0
    • +37
    ./Modes/FPModes/ModeGenPermeateDefeatured.c
    • -1
    • +1
    ./Modes/FPModes/ModeGenPermeateDefeatured.h
    • -0
    • +62
    ./Modes/FPModes/ModePreGenPermeate.c
    • -7
    • +8
    ./Modes/FPModes/ModePreGenPermeate.h
    • -0
    • +41
    ./Modes/FPModes/ModePreGenPermeateDefeatured.c
    • -1
    • +1
    ./Modes/FPModes/ModePreGenPermeateDefeatured.h
    • -0
    • +1
    ./Services/Messaging.c
12 hrs ago

Vendor - TEL - Sameer Poyil

LDT-2004 Dry bicart fixing merge issue

Options
    • -5
    • +6
    ./Services/Messaging.c
Thursday 05 Mar
7:52 pm

Vendor - TEL - Sameer Poyil

All comments should start with capital letter

All comments should start with capital letter

7:37 pm

Vendor - TEL - Sameer Poyil

comment alignment issue

comment alignment issue

7:37 pm

Vendor - TEL - Sameer Poyil

it should be mentioned what function is used for calculating the equation.

it should be mentioned what function is used for calculating the equation.

7:30 pm

Vendor - TEL - Sameer Poyil

above comment applicable here .

above comment applicable here .

7:29 pm

Vendor - TEL - Sameer Poyil

if sensor id is not in the finite state enum value , it return 0 and here it calculate for P9

if sensor id is not in the finite state enum value , it return 0 and here it calculate for P9

7:21 pm

Vendor - TEL - Sameer Poyil

parameter is sensorId, not sensor

parameter is sensorId, not sensor

5:12 pm

Dara Navaei

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

Options
    • -1
    • +1
    ./DDCommon.h
5:12 pm

Dara Navaei

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

Options
    • -2
    • +2
    ./Monitors/BloodLeak.c
5:10 pm

Dara Navaei

4:36 pm

Vendor - TEL - Jashwant Gantyada and Sean Nash

4:25 pm

Dara Navaei

3:07 pm

Dara Navaei

LDT-3605 added code in the blood leak driver to reset its state machine once the hardware configuration has changed

Options
    • -3
    • +16
    ./Monitors/BloodLeak.c
3:02 pm

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3605_(DD) BLD sensor intermittently does not read for B2.0 HW
LEAHI-DD-FIRMWARE-LDT-3605_(DD) BLD sensor intermittently does not read for B2.0 HW
1:29 pm

Sean Nash

1:20 pm

Sean Nash

Add default. What should we do if caller passed invalid sensor ID?

Add default. What should we do if caller passed invalid sensor ID?

11:42 am

Vendor - TEL - Jashwant Gantyada

LDT-3103 special build changes

Options
    • -4
    • +4
    ./Modes/ModeGenDialysate.c
9:46 am

Michael Garthwaite

updated newCondModel to be a U32 before check then casted to enum after. Note: Calculate conductivity has a default statement that will catch any value that is outside of the model enum ( and execu...

updated newCondModel to be a U32 before check then casted to enum after. Note: Calculate conductivity has a default statement that will catch any value that is outside of the model enum ( and execute standard conductivity model )

9:39 am

Michael Garthwaite

LDT-3372: fixes set conductivity model per code review

Options
    • -2
    • +2
    ./Drivers/ConductivityTeensy.c
7:32 am

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3603_Ability to set operation sub mode on IOFP sub system
LEAHI-DD-FIRMWARE-LDT-3603_Ability to set operation sub mode on IOFP sub system
Wednesday 04 Mar
2:13 pm

Vendor - TEL - Sameer Poyil

Then need to add a default case in that function

Then need to add a default case in that function

2:10 pm

Vendor - TEL - Sameer Poyil

If all enumeration constants are non-negative and fit within the range of an int or unsigned int, the compiler has the flexibility to choose either a signed or unsigned type. In many common impleme...

If all enumeration constants are non-negative and fit within the range of an int or unsigned int, the compiler has the flexibility to choose either a signed or unsigned type. In many common implementations, it defaults to a signed int

2:05 pm

Sean Nash

Looked better before - now misspelled.

Looked better before - now misspelled.

2:04 pm

Sean Nash

0.0F

0.0F

2:04 pm

Sean Nash

indent looks 1 space off.

indent looks 1 space off.

2:01 pm

Sean Nash

If we include math, I like to see comment explaining what we're doing with it. Why remove it?

If we include math, I like to see comment explaining what we're doing with it. Why remove it?

2:00 pm

Sean Nash

Declare at top of scope, assign down here if needed.

Declare at top of scope, assign down here if needed.

1:59 pm

Sean Nash

If there is only one target temperature, why are we passing in a heater ID to function? This seems weird to me having both heaters share a target.

If there is only one target temperature, why are we passing in a heater ID to function? This seems weird to me having both heaters share a target.