td-firmware / firmware

App

Clone Tools

Stats

Commits this week: 33
Total Committers: 16
Last Commit: 12:26
Commits this week: 33

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 29 May
3:25 pm

Dara Navaei

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

Options
    • -1
    • +1
    ./TDCommon.h
3:24 pm

Dara Navaei

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

Options
    • -2
    • +2
    ./Modes/ModePreTreat.c
    • -2
    • +2
    ./Modes/ModeTreatment.c
    • -2
    • +2
    ./Modes/ModeTreatment.h
    • -2
    • +2
    ./Monitors/Pressures.c
    • -2
    • +2
    ./Monitors/Pressures.h
    • -2
    • +2
    ./Services/Messaging.c
    • -2
    • +2
    ./Services/Messaging.h
    • -2
    • +2
    ./Services/TxParams.c
    • -2
    • +2
    ./Services/TxParams.h
2:57 pm

Sean Nash and Vendor - TEL - Arpita Srivastava

2:43 pm

Vendor - TEL - Praneeth Bunne

LDT-4013: Address CR comments

Options
    • -9
    • +17
    ./Modes/StateTxDialysis.c
    • -6
    • +8
    ./Modes/StateTxPaused.c
    • -2
    • +2
    ./Services/FluidBolus.c
2:43 pm

Vendor - TEL - Vijay Pamula

LDT-3970: Addressed review comments for edit treatment parameters

Options
    • -1
    • +1
    ./Monitors/Pressures.c
2:33 pm

Vendor - TEL - Arpita Srivastava

Please add the min and max for tmp as well in the outputs.

Please add the min and max for tmp as well in the outputs.

2:25 pm

Sean Nash

rejReason = INVALID REQUEST FORMAT

rejReason = INVALID REQUEST FORMAT

12:11 pm

Vendor - TEL - Vijay Pamula

LDT-3970: Merge latest staging and Addressed Review comments

Options
    • -0
    • +1
    ./Services/Messaging.c
    • -0
    • +1
    ./Services/Messaging.h
    • -2
    • +10
    ./Services/TxParams.c
10:50 am

Vendor - TEL - Varshini Nagabooshanam

Fixes

Options
    • -0
    • +1
    ./Services/StateServices/DrySelfTests.c
    • -0
    • +1
    ./Services/StateServices/DrySelfTests.h
9:58 am

Sean Nash

Why would change request from UI come in separate messages (1 for duration and 1 for volume)?

Why would change request from UI come in separate messages (1 for duration and 1 for volume)?

9:56 am

Sean Nash

How can settings be pending when you set that flag to TRUE in the code below this? And why would we want to use uf volume setting in the else part of this ternary?

How can settings be pending when you set that flag to TRUE in the code below this? And why would we want to use uf volume setting in the else part of this ternary?

9:52 am

Sean Nash

else reason is Invalid Request Format

else reason is Invalid Request Format

9:47 am

Sean Nash

Should check if iso UF volume + UF volume < max. And if user can do multiple iso uf sessions, you would check iso UF volume + any previous iso UF volume + UF volume. I don't know if there is a mini...

Should check if iso UF volume + UF volume < max. And if user can do multiple iso uf sessions, you would check iso UF volume + any previous iso UF volume + UF volume.
I don't know if there is a minimum iso UF volume.

9:45 am

Sean Nash

Looking for "less than current" only makes sense if user can do multiple iso UF sessions in a treatment. And if that's the case, we should be looking at how much time we've already spent doing iso ...

Looking for "less than current" only makes sense if user can do multiple iso UF sessions in a treatment. And if that's the case, we should be looking at how much time we've already spent doing iso UF instead of how much time we've spent in the HD or HDF treatment.

9:40 am

Sean Nash

I think we have to check whether the iso uf duration + treatment duration does not exceed maximum treatment duration. And I'm not sure if there is a minimum duration for iso uf, but if there is a m...

I think we have to check whether the iso uf duration + treatment duration does not exceed maximum treatment duration.
And I'm not sure if there is a minimum duration for iso uf, but if there is a minimum, it probably isn't the same as the minimum treatment duration.

9:37 am

Sean Nash

Just add this as an AND condition in the if statement above.

Just add this as an AND condition in the if statement above.

9:33 am

Sean Nash

If we are in treatment end state, the treatment is completed (no need to check that) and I don't think iso UF is a modality.

If we are in treatment end state, the treatment is completed (no need to check that) and I don't think iso UF is a modality.

9:31 am

Sean Nash

Do we need this? Check with Raghu. I don't think iso UF is a modality (at least not yet).

Do we need this? Check with Raghu. I don't think iso UF is a modality (at least not yet).

9:19 am

Sean Nash

Align all or none of the =

Align all or none of the =

9:14 am

Sean Nash

I don't think iso uf is considered a modality - at least not in phase 1 of development.

I don't think iso uf is considered a modality - at least not in phase 1 of development.

Thursday 28 May
10:43 pm

Sean Nash

I think these remaining 4 else ifs need to have another && condition that your service is in idle state so that we don't change back into one of these 4 states while a bolus is in progress.

I think these remaining 4 else ifs need to have another && condition that your service is in idle state so that we don't change back into one of these 4 states while a bolus is in progress.

10:33 pm

Sean Nash

Can we restore the real code here yet? Ask Dara.

Can we restore the real code here yet? Ask Dara.

4:55 pm

Sean Nash

Why isn't this TRUE == ?

Why isn't this TRUE == ?

4:35 pm

Vendor - TEL - Praneeth Bunne

LDT-4013: Update code related to fluid bolus

Options
    • -10
    • +5
    ./Modes/StateTxBloodPrime.c
    • -11
    • +5
    ./Modes/StateTxDialysis.c
    • -39
    • +31
    ./Modes/StateTxPaused.c
    • -31
    • +31
    ./Services/AlarmMgmtTD.h
    • -8
    • +2
    ./Services/FluidBolus.c
    • -1
    • +1
    ./Services/FluidBolus.h
4:18 pm

Vendor - TEL - Jashwant Gantyada

LDT-4009 Isolated UF implementation

Options
    • -0
    • +424
    ./Modes/StateTxIsolatedUF.c
    • -0
    • +60
    ./Modes/StateTxIsolatedUF.h
4:07 pm

Vendor - TEL - Jashwant Gantyada

LDT-4009 Isolated UF TD implementation

Options
    • -28
    • +350
    ./Modes/ModeTreatment.c
    • -0
    • +16
    ./Modes/ModeTreatment.h
    • -1
    • +5
    ./Services/Messaging.c
    • -25
    • +325
    ./Services/TxParams.c
    • -8
    • +49
    ./Services/TxParams.h
1:47 pm

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-3975_Water Treatment and Degassing - FW Iteration - 4/4:
LEAHI-TD-FIRMWARE-LDT-3975_Water Treatment and Degassing - FW Iteration - 4/4:
1:41 pm

Sean Nash

LDT-3975: Added s/w fault IDs to distinguish PID control s/w faults.

Options
    • -0
    • +5
    ./Services/AlarmMgmtSWFaults.h
12:49 pm

Sean Nash

use ternary to make TRUE or FALSE.

use ternary to make TRUE or FALSE.

11:45 am

Vendor - TEL - Varshini Nagabooshanam

Fixed

Fixed