td-firmware / firmware / App / Controllers

BloodFlow.c

Clone Tools

Stats

Commits this week: 2
Total Committers: 5
Last Commit: 21 Apr
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
Tuesday 21 Apr
3:27 pm

Vendor - TEL - Jashwant Gantyada

updated.

updated.

3:26 pm

Vendor - TEL - Jashwant Gantyada

done.

done.

3:26 pm

Vendor - TEL - Jashwant Gantyada

updated.

updated.

3:26 pm

Vendor - TEL - Jashwant Gantyada

updated.

updated.

3:26 pm

Vendor - TEL - Jashwant Gantyada

removed and used default value as they are the same.

removed and used default value as they are the same.

3:26 pm

Vendor - TEL - Jashwant Gantyada

yes. updated.

yes. updated.

3:26 pm

Vendor - TEL - Jashwant Gantyada

updated and used as suggested.

updated and used as suggested.

3:25 pm

Vendor - TEL - Jashwant Gantyada

noted.

noted.

1:28 pm

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?

1:26 pm

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?

1:25 pm

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.

1:24 pm

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.

1:17 pm

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.

12:58 pm

Sean Nash

We typically refer to set blood flow rate as Qb.

We typically refer to set blood flow rate as Qb.

12:57 pm

Sean Nash

Refactor this function to have only one return statement.

Refactor this function to have only one return statement.

9:24 am

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

2:04 pm

Dara Navaei

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

Options
    • -2
    • +2
    ./BloodFlow.c
  2. … 2 more files in changeset.
1:58 pm

Vendor - TEL - Jashwant Gantyada

1:57 pm

Vendor - TEL - Jashwant Gantyada

Tuesday 17 Mar
2:15 pm

Sean Nash

1:05 pm

Vendor - TEL - Varshini Nagabooshanam

LDT-3126-blood-prime-td-updates

Options
    • -8
    • +8
    ./BloodFlow.c
  2. … 1 more file in changeset.
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.