td-firmware / firmware / App / Controllers

AirPump.c

Clone Tools

Stats

Commits this week: 1
Total Committers: 7
Last Commit: 03 Aug
Commits this week: 1

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
Wednesday 05 Aug
12:45 pm

Sean Nash

What if we're in paused state (because your alarm has stop property set)? Should we return TRUE here if in paused state and interrupted state was the await syringe load confirmation state?

What if we're in paused state (because your alarm has stop property set)? Should we return TRUE here if in paused state and interrupted state was the await syringe load confirmation state?

12:40 pm

Sean Nash

Right now, the load syringe alarm indicates stop so we will go to paused state. Is that what we want to do? Don't we want to stay in this state and wait for confirmation?

Right now, the load syringe alarm indicates stop so we will go to paused state. Is that what we want to do? Don't we want to stay in this state and wait for confirmation?

12:37 pm

Sean Nash

Should this alarm have stop property set to FALSE?

Should this alarm have stop property set to FALSE?

Tuesday 04 Aug
11:10 am

Vendor - TEL - Arpita Srivastava

Can we please break this line as to match our standard of 100 columns? SET_ALARM_WITH_2_U32_DATA( ALARM_ID_TD_AIR_PUMP_STALL, currentAirPumpPowerLevel, getU32OverrideValue( &currentAirPumpRPM ) );

Can we please break this line as to match our standard of 100 columns?

SET_ALARM_WITH_2_U32_DATA( ALARM_ID_TD_AIR_PUMP_STALL,
currentAirPumpPowerLevel,
getU32OverrideValue( &currentAirPumpRPM ) );

Thursday 30 Jul
10:31 am

Sean Nash

Set "Stops" property back to TRUE like the other reserved UI alarms.

Set "Stops" property back to TRUE like the other reserved UI alarms.

Wednesday 29 Jul
3:04 pm

Vinayakam Mani

2:35 pm

Vendor - TEL - Santhoshkumar Reddy Alasandagutti

Updated

Updated

Tuesday 28 Jul
2:47 pm

Vendor - TEL - Arpita Srivastava

break this line

break this line

2:39 pm

Vendor - TEL - Arpita Srivastava

add an empty else block after the last else if block in these functions handlePreTxHeparinPreloadState handlePreTxHeparinAwaitSyringeLoadConfirmationState handlePreTxHeparinSeekState handlePreTxHep...

add an empty else block after the last else if block in these functions
handlePreTxHeparinPreloadState
handlePreTxHeparinAwaitSyringeLoadConfirmationState
handlePreTxHeparinSeekState
handlePreTxHeparinPrimeState
handlePreTxHeparinBolusState

2:32 pm

Vendor - TEL - Arpita Srivastava

break this line

break this line

10:44 am

Vendor - TEL - Arpita Srivastava

Add initialization for heparinSetupResumeRequested and syringeLoadConfirmed

Add initialization for heparinSetupResumeRequested and syringeLoadConfirmed

10:42 am

Vendor - TEL - Arpita Srivastava

Align all the function comments

Align all the function comments

10:19 am

Vendor - TEL - Arpita Srivastava

Delete this row.

Delete this row.

Monday 27 Jul
5:09 pm

Vinayakam Mani

the function call seems returning U16. Type cast to U32.

the function call seems returning U16. Type cast to U32.

Sunday 26 Jul
8:37 pm

Sean Nash

Should this be the "Load Syringe" alarm that you just added?

Should this be the "Load Syringe" alarm that you just added?

Tuesday 21 Jul
5:45 pm

Sean Nash

I think ( setTxDur - preStop ) should be changed to ( preStop ). Also, I think preStop should be renamed to hepDurMin.

I think ( setTxDur - preStop ) should be changed to ( preStop ).
Also, I think preStop should be renamed to hepDurMin.

2:10 pm

Sean Nash

Can we remote this TODO comment?

Can we remote this TODO comment?

2:09 pm

Sean Nash

Remove blank line.

Remove blank line.

2:08 pm

Sean Nash

We should go to dry self test state. Why did this change to Rx state?

We should go to dry self test state. Why did this change to Rx state?

2:07 pm

Sean Nash

Comment - should be "install" not "instll".

Comment - should be "install" not "instll".

Monday 20 Jul
9:02 am

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-5214_[FW+Dialin] New override for Air Pump Measured Speed parameter
LEAHI-TD-FIRMWARE-LDT-5214_[FW+Dialin] New override for Air Pump Measured Speed parameter
Thursday 16 Jul
3:02 pm

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-3992_Disposable Tube Set Auto Priming Process - Saline - FW Implementation - 1/6:
LEAHI-TD-FIRMWARE-LDT-3992_Disposable Tube Set Auto Priming Process - Saline - FW Implementation - 1/6:
Tuesday 14 Jul
11:17 pm

Sean Nash

I don't see any change here. How does this function get called? I don't see the function being used anywhere - nobody is calling it. I think it should be called by Messaging.c when a UI confirmatio...

I don't see any change here. How does this function get called? I don't see the function being used anywhere - nobody is calling it. I think it should be called by Messaging.c when a UI confirmation msg is received.

5:53 pm

Sean Nash

171

171

Monday 13 Jul
1:47 pm

Sean Nash

Who is calling this function? Should be coming from UI, right? So should be a message handler (returns BOOL, takes a message as param, function is in table at top of Messaging.c).

Who is calling this function? Should be coming from UI, right? So should be a message handler (returns BOOL, takes a message as param, function is in table at top of Messaging.c).

1:22 pm

Sean Nash

Need to set state to preload state I think. Otherwise you will go to interrupted state.

Need to set state to preload state I think. Otherwise you will go to interrupted state.

12:44 pm

Sean Nash

Why did you remote call to init function here? Is it initialized somewhere else?

Why did you remote call to init function here? Is it initialized somewhere else?

12:41 pm

Sean Nash

Use ternary operation to assign TRUE or FALSE explicitly.

Use ternary operation to assign TRUE or FALSE explicitly.

12:39 pm

Sean Nash

This is at end of enum list above. So it should be at end of table here.

This is at end of enum list above. So it should be at end of table here.

Thursday 09 Jul
2:07 pm

Sean Nash

What alarm is this? It doesn't look like you've added a new alarm. This is one of the reserved (unused) alarms that UI s/w might use.

What alarm is this? It doesn't look like you've added a new alarm. This is one of the reserved (unused) alarms that UI s/w might use.