dd-firmware / firmware / App / Controllers

DryBiCart.c

Clone Tools

Stats

Commits this week: 15
Total Committers: 6
Last Commit: 18 Jun
Commits this week: 15

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
Thursday 18 Jun
3:18 pm

Vendor - TEL - Sameer Poyil

LDT-4612 drybicart updates

Options
    • -2
    • +2
    ./DryBiCart.c
2:34 pm

Vendor - TEL - Sameer Poyil

LDT-4612 persistence on D66 updates

Options
    • -11
    • +8
    ./DryBiCart.c
1:53 pm

Vinayakam Mani

Looks On event handled later. One event should be fine for On condtion.

Looks On event handled later. One event should be fine for On condtion.

1:51 pm

Vinayakam Mani

Space after 0

Space after 0

1:48 pm

Vinayakam Mani

prevDryBicartState

prevDryBicartState

1:46 pm

Vinayakam Mani

I guess, you should use OnOff status as second argument, since isBalChamberSwitchingActive varirable set after this function, it won't have latest status.

I guess, you should use OnOff status as second argument, since isBalChamberSwitchingActive varirable set after this function, it won't have latest status.

1:31 pm

Vinayakam Mani

prevBalChamberState?

prevBalChamberState?

10:21 am

Vendor - TEL - Arpita Srivastava

Should we also send an event when we start the heater?

Should we also send an event when we start the heater?

10:11 am

Vendor - TEL - Arpita Srivastava

We can combine all these condition checks in a single if() at line 423 separated by || and then we can send the event change in one line. The code will still work if we remove the first condition c...

We can combine all these condition checks in a single if() at line 423 separated by || and then we can send the event change in one line.
The code will still work if we remove the first condition check for prevState from line 427 and 431

10:06 am

Vendor - TEL - Sameer Poyil

LDT-4409 fix for concentrate pump not running

Options
    • -3
    • +0
    ./DryBiCart.c
  2. … 2 more files in changeset.
4:02 am

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-4612_Dialysate Composition (Dry Bicarbonate) - FW Iteration - 4/1:
LEAHI-DD-FIRMWARE-LDT-4612_Dialysate Composition (Dry Bicarbonate) - FW Iteration - 4/1:
Wednesday 17 Jun
10:13 pm

Vendor - TEL - Sameer Poyil

LDT-4612 fix for micing control integration issues

Options
    • -21
    • +18
    ./DryBiCart.c
  2. … 1 more file in changeset.
Tuesday 16 Jun
1:37 pm

Dara Navaei

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

Options
    • -1
    • +1
    ./DryBiCart.c
  2. … 13 more files in changeset.
1:24 pm

Vendor - TEL - Jashwant Gantyada and Vinayakam Mani

1:23 pm

Vendor - TEL - Sameer Poyil

LDT-4409 addressed review comments

Options
    • -2
    • +2
    ./DryBiCart.c
  2. … 3 more files in changeset.
1:16 pm

Vendor - TEL - Jashwant Gantyada

11:55 am

Vinayakam Mani

Michaels latest code resolved this issue. make sure when you merge to staging, pull the latest from staging. Check with Michael on code commit in staging.

Michaels latest code resolved this issue. make sure when you merge to staging, pull the latest from staging. Check with Michael on code commit in staging.

11:09 am

Vendor - TEL - Sameer Poyil

LDT-4409 addressed review comments

Options
    • -10
    • +4
    ./DryBiCart.c
  2. … 5 more files in changeset.
10:33 am

Vendor - TEL - Sameer Poyil

I will mark it as TODO , so you can close it

I will mark it as TODO , so you can close it

10:24 am

Vinayakam Mani

Update name.

Update name.

10:22 am

Vinayakam Mani

Add a Todo comment to resolve the issue.

Add a Todo comment to resolve the issue.

10:20 am

Vinayakam Mani

Align the indentation.

Align the indentation.

10:17 am

Vinayakam Mani

Align the indentation.

Align the indentation.

10:15 am

Vinayakam Mani

Do we need these comments? delete it if not required.

Do we need these comments? delete it if not required.

10:11 am

Vinayakam Mani

Align the indent

Align the indent

7:02 am

Vendor - TEL - Sameer Poyil

LDT-4409 addressed review comments

Options
    • -16
    • +11
    ./DryBiCart.c
  2. … 6 more files in changeset.
Monday 15 Jun
2:17 pm

Vinayakam Mani

Remove empty line

Remove empty line

2:16 pm

Vinayakam Mani

Remove unused header files.

Remove unused header files.

2:15 pm

Vinayakam Mani

Remove empty line.

Remove empty line.

2:14 pm

Vinayakam Mani

can't we use DialsysateConductivity - local variable instead of getTargetCond()?

can't we use DialsysateConductivity - local variable instead of getTargetCond()?