dd-firmware / firmware / App

Controllers

Clone Tools

Stats

Commits this week: 25
Total Committers: 15
Last Commit: 15 Jun
Commits this week: 25

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
Thursday 11 Jun
9:50 am

Michael Garthwaite

LDT-4001: code review updates

Options
    • -3
    • +4
    ./SubstitutionPump.c
    • -1
    • +1
    ./SubstitutionPump.h
9:43 am

Sean Nash

Change speed to rate.

Change speed to rate.

9:43 am

Sean Nash

pumpTargetRate

pumpTargetRate

9:43 am

Sean Nash

Change function name to say rate instead of speed

Change function name to say rate instead of speed

9:42 am

Sean Nash

Is this a rate too?

Is this a rate too?

9:42 am

Sean Nash

Is this a rate too?

Is this a rate too?

9:40 am

Sean Nash

Change speed to rate.

Change speed to rate.

9:40 am

Sean Nash

Is currentPumpSpeed actually a rate?

Is currentPumpSpeed actually a rate?

9:38 am

Sean Nash

Is this comment correct? Seems like we're stopping pump if requested to turn off.

Is this comment correct? Seems like we're stopping pump if requested to turn off.

9:36 am

Sean Nash

Is currentPumpSpeed also a rate? If not, we're subtracting a speed from a rate which doesn't make much sense.

Is currentPumpSpeed also a rate? If not, we're subtracting a speed from a rate which doesn't make much sense.

9:34 am

Sean Nash

Align comment.

Align comment.

9:34 am

Sean Nash

Align comment. Change "speed" to "rate in comment too. Also, I don't see you initializing these rates.

Align comment. Change "speed" to "rate in comment too.
Also, I don't see you initializing these rates.

9:33 am

Michael Garthwaite

fixed thanks

fixed thanks

9:30 am

Michael Garthwaite

LDT-4001: code review updates

Options
    • -24
    • +24
    ./SubstitutionPump.c
    • -2
    • +2
    ./SubstitutionPump.h
9:20 am

Sean Nash

Change to pumpTargetRate since mL/min is a rate.

Change to pumpTargetRate since mL/min is a rate.

Wednesday 10 Jun
4:30 pm

Vendor - TEL - Arpita Srivastava

LDT-3735: Split NVMsgQ into NVJobQ and NVMessaging

Options
    • -7
    • +7
    ./DialysatePumps.c
  2. … 14 more files in changeset.
1:20 pm

Michael Garthwaite

fixed thanks

fixed thanks

1:20 pm

Michael Garthwaite

fixed thanks

fixed thanks

1:20 pm

Michael Garthwaite

fixed thanks

fixed thanks

1:18 pm

Vendor - TEL - Jashwant Gantyada

1:18 pm

Michael Garthwaite

LDT-4001: code review comments

Options
    • -2
    • +2
    ./Heaters.c
  2. … 1 more file in changeset.
9:22 am

Vendor - TEL - Sameer Poyil

Please check with TD, I have seen one of the develop branch this populated in pre gen and gen.

Please check with TD, I have seen one of the develop branch this populated in pre gen and gen.

Tuesday 09 Jun
10:29 pm

Vinayakam Mani

why TODO?

why TODO?

10:24 pm

Vinayakam Mani

Do we need to close D64 as well?

Do we need to close D64 as well?

10:23 pm

Vinayakam Mani

Delete the commented line

Delete the commented line

10:17 pm

Vinayakam Mani

handleBicarbChamberInitiateSupply?

handleBicarbChamberInitiateSupply?

10:15 pm

Vinayakam Mani

Update this.

Update this.

10:14 pm

Vinayakam Mani

Remove this comment.

Remove this comment.

10:13 pm

Vinayakam Mani

Update Inputs and Outputs throughout the file.

Update Inputs and Outputs throughout the file.

10:06 pm

Vinayakam Mani

Do we need this?

Do we need this?