td-firmware / firmware / App / Modes

ModeTreatment.c

Clone Tools

Stats

Commits this week: 8
Total Committers: 8
Last Commit: 22 May
Commits this week: 8

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
Wednesday 27 May
4:42 pm

Vendor - TEL - Arpita Srivastava

Which alarm?

Which alarm?

4:42 pm

Vendor - TEL - Arpita Srivastava

bpPublishTimerCtr is the output?

bpPublishTimerCtr is the output?

4:40 pm

Vendor - TEL - Arpita Srivastava

Input is none. Add @Alarms

Input is none.
Add @Alarms

4:38 pm

Vendor - TEL - Arpita Srivastava

Should we list the alarm ids? @Alarms?

Should we list the alarm ids? @Alarms?

4:36 pm

Vendor - TEL - Arpita Srivastava

requestAbortBPReading is the output?

requestAbortBPReading is the output?

4:36 pm

Vendor - TEL - Arpita Srivastava

requestPedsBPReading is the output?

requestPedsBPReading is the output?

4:35 pm

Vendor - TEL - Arpita Srivastava

requestAdultBPReading is the output?

requestAdultBPReading is the output?

4:35 pm

Vendor - TEL - Arpita Srivastava

pendingVitalsRequest is the output?

pendingVitalsRequest is the output?

4:34 pm

Vendor - TEL - Arpita Srivastava

We should initialize in the init function?

We should initialize in the init function?

4:31 pm

Vendor - TEL - Arpita Srivastava

bpMeasurementReady is an output?

bpMeasurementReady is an output?

4:29 pm

Vendor - TEL - Arpita Srivastava

nextState is the return value. Should we add requestAdultBPMeasurement, bpMeasurementReady and bpDriverError to outputs?

nextState is the return value.
Should we add requestAdultBPMeasurement, bpMeasurementReady and bpDriverError to outputs?

4:24 pm

Vendor - TEL - Arpita Srivastava

Where are we updating the bpResults?

Where are we updating the bpResults?

4:22 pm

Vendor - TEL - Arpita Srivastava

We should initialize this in init functions.

We should initialize this in init functions.

4:22 pm

Vendor - TEL - Arpita Srivastava

please add a comment

please add a comment

4:18 pm

Vendor - TEL - Arpita Srivastava

pressure is a param?

pressure is a param?

4:18 pm

Vendor - TEL - Arpita Srivastava

command is a param?

command is a param?

3:21 pm

Sean Nash

Move to driver.

Move to driver.

3:21 pm

Sean Nash

Recommend command be a U08 here. Driver should have the enum and pass the enum to this function. FPGA should just take a byte.

Recommend command be a U08 here. Driver should have the enum and pass the enum to this function. FPGA should just take a byte.

3:09 pm

Sean Nash

Move publish call from exec to the check state handle function.

Move publish call from exec to the check state handle function.

3:07 pm

Sean Nash

I don't think we need this.

I don't think we need this.

3:07 pm

Sean Nash

Don't need a flag here. Need a flag in driver and also an isBPCuffConnected() function in driver that we can call from this controller.

Don't need a flag here. Need a flag in driver and also an isBPCuffConnected() function in driver that we can call from this controller.

3:02 pm

Sean Nash

Remove.

Remove.

3:00 pm

Sean Nash

Just use requestAdultBPReading.

Just use requestAdultBPReading.

2:57 pm

Sean Nash

Remove since we're only publishing when we have new measurements.

Remove since we're only publishing when we have new measurements.

1:22 pm

Sean Nash

Use the NEARLY_ZERO constant (instead 0f 0.01F) like the other one.

Use the NEARLY_ZERO constant (instead 0f 0.01F) like the other one.

1:07 pm

Sean Nash

Add a doxygen comment above this array.

Add a doxygen comment above this array.

1:06 pm

Sean Nash

Add a doxygen comment above this array (similar to structures).

Add a doxygen comment above this array (similar to structures).

1:04 pm

Sean Nash

Add a doxygen comment above structure for the whole structure (see structure above for example).

Add a doxygen comment above structure for the whole structure (see structure above for example).

1:03 pm

Sean Nash

Add a doxygen comment for this definition.

Add a doxygen comment for this definition.

Tuesday 26 May
11:30 am

Raghu Kallala

remove camel case for SetPoint, instead use Setpoint Update line 659 too

remove camel case for SetPoint, instead use Setpoint
Update line 659 too