td-firmware / firmware / App / Controllers

BloodFlow.c

Clone Tools

Stats

Commits this week: 2
Total Committers: 5
Last Commit: 06 Mar
Commits this week: 2

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
Today
1 hr ago

Vendor - TEL - Jashwant Gantyada

updated.

updated.

1 hr ago

Vendor - TEL - Jashwant Gantyada

done.

done.

1 hr ago

Vendor - TEL - Jashwant Gantyada

updated.

updated.

1 hr ago

Vendor - TEL - Jashwant Gantyada

updated.

updated.

1 hr ago

Vendor - TEL - Jashwant Gantyada

removed and used default value as they are the same.

removed and used default value as they are the same.

1 hr ago

Vendor - TEL - Jashwant Gantyada

yes. updated.

yes. updated.

1 hr ago

Vendor - TEL - Jashwant Gantyada

updated and used as suggested.

updated and used as suggested.

1 hr ago

Vendor - TEL - Jashwant Gantyada

noted.

noted.

3 hrs ago

Sean Nash

We have a reset function above that does this. Should we call it?

We have a reset function above that does this. Should we call it?

3 hrs ago

Sean Nash

Are we filtering at zero set rate? I guess we should, so then the duration would be 1 sec then?

Are we filtering at zero set rate? I guess we should, so then the duration would be 1 sec then?

3 hrs ago

Sean Nash

I think minimum samples would be for max rate (500 mL/min), so 3 secs at 20 Hz = 60 samples.

I think minimum samples would be for max rate (500 mL/min), so 3 secs at 20 Hz = 60 samples.

3 hrs ago

Sean Nash

PRODUCT should probably come before FILTER_SAMPLES above so that it can be used in that definition. Show derivation again. i.e. (500 * 3) indicating we want 3 seconds of filter duration for 500 mL/...

PRODUCT should probably come before FILTER_SAMPLES above so that it can be used in that definition.
Show derivation again. i.e. (500 * 3) indicating we want 3 seconds of filter duration for 500 mL/min rate and duration grows as rate is reduced.

3 hrs ago

Sean Nash

600 will take a lot of memory (600 x 4 bytes = 2.4K). Ok for now, but if memory gets scarce, we will have to consider sampling less frequently to reduce the size of the moving average array. Also,...

600 will take a lot of memory (600 x 4 bytes = 2.4K). Ok for now, but if memory gets scarce, we will have to consider sampling less frequently to reduce the size of the moving average array.

Also, show derivation of this number so it's clear what it means and how we got to 600.
i.e. ((PRODUCT / 50) * (MS_PER_SECOND / TASK_GENERAL_INTERVAL)) indicating how many samples we need to buffer for worst case which is lowest flow rate (50) requiring max duration (30 sec) at 20 Hz sample rate.

4 hrs ago

Sean Nash

We typically refer to set blood flow rate as Qb.

We typically refer to set blood flow rate as Qb.

4 hrs ago

Sean Nash

Refactor this function to have only one return statement.

Refactor this function to have only one return statement.

7 hrs ago

Sean Nash

You could just do a memset() here.

You could just do a memset() here.

Thursday 16 Apr
2:47 pm

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-3959_Blood Flow Rate - FW Implementation - 1/1:
LEAHI-TD-FIRMWARE-LDT-3959_Blood Flow Rate - FW Implementation - 1/1:
Friday 27 Mar
3:35 pm

Vendor - TEL - Varshini Nagabooshanam

1:58 pm

Vendor - TEL - Jashwant Gantyada

1:57 pm

Vendor - TEL - Jashwant Gantyada

Tuesday 17 Mar
2:15 pm

Sean Nash

9:58 am

Sean Nash

Wrong place.

Wrong place.

9:57 am

Sean Nash

I'll drop by and we can set properties together.

I'll drop by and we can set properties together.

Monday 16 Mar
5:22 pm

Vendor - TEL - Varshini Nagabooshanam

Added

Added

5:22 pm

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

5:22 pm

Vendor - TEL - Varshini Nagabooshanam

Sorry, Updated

Sorry, Updated

5:21 pm

Vendor - TEL - Varshini Nagabooshanam

Updated, if this is wrong then I need your help

Updated, if this is wrong then I need your help

4:33 pm

Sean Nash

Match properties from Denali where you can. The priorities should both be HIGH. The rank for the warning should be a higher number. The alarm should be TRUE for no resume and no rinseback. So, I kn...

Match properties from Denali where you can. The priorities should both be HIGH. The rank for the warning should be a higher number. The alarm should be TRUE for no resume and no rinseback. So, I know these properties aren't correct.
One thing I don't think we should copy from Denali though - I think the warning should not be TRUE for the STOP property so that we stay in blood prime state.

4:31 pm

Sean Nash

Add a blank line after declaration.

Add a blank line after declaration.

4:30 pm

Sean Nash

I think we should just have == here so that we only warn them once.

I think we should just have == here so that we only warn them once.