leahi-common

Clone Tools

Stats

Commits this week: 12
Total Committers: 16
Last Commit: 14:16
Commits this week: 12

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 27 Feb
10:04 am

Vendor - TEL - Jashwant Gantyada

9:17 am

Vendor - TEL - Sameer Poyil

pushed new code

pushed new code

9:14 am

Vendor - TEL - Sameer Poyil

Fixed

Fixed

9:14 am

Vendor - TEL - Sameer Poyil

fixed it in commit ID 71a6797

fixed it in commit ID 71a6797

8:48 am

Sean Nash

Has integrity unit been added to TD project?

Has integrity unit been added to TD project?

8:44 am

Sean Nash

Where are these being triggered?

Where are these being triggered?

8:32 am

Sean Nash

Is this necessary? If so, why?

Is this necessary? If so, why?

Thursday 26 Feb
11:24 pm

Sean Nash

Response payload is not U08. I believe UI will be expecting a payload of type UI_RESPONSE_PAYLOAD_T (defined in Messaging.h). Verify with Nico that this is the payload he is expecting in response. ...

Response payload is not U08. I believe UI will be expecting a payload of type UI_RESPONSE_PAYLOAD_T (defined in Messaging.h). Verify with Nico that this is the payload he is expecting in response. See signalPauseResumeUF() function for example of how to respond to a UI request.

11:24 pm

Sean Nash

Same comment as other message handling function. Use correct payload type. See signalPauseResumeUF() function for example of how to respond to a UI request.

Same comment as other message handling function. Use correct payload type. See signalPauseResumeUF() function for example of how to respond to a UI request.

11:19 pm

Sean Nash

Why do we need this local variable when we have the static requestedBloodFlowRate_mL_min variable?

Why do we need this local variable when we have the static requestedBloodFlowRate_mL_min variable?

11:18 pm

Sean Nash

Add a blank line after declarations.

Add a blank line after declarations.

11:18 pm

Sean Nash

I would say "TRUE if request is accepted, FALSE if rejected."

I would say "TRUE if request is accepted, FALSE if rejected."

11:16 pm

Sean Nash

I would say "TRUE if request is accepted, FALSE if rejected."

I would say "TRUE if request is accepted, FALSE if rejected."

11:11 pm

Sean Nash

We do not always want to return to run state. If we were in ramp state prior to pause state, we would want to go back to ramp state here. So you need to remember your previous state in a new variable.

We do not always want to return to run state. If we were in ramp state prior to pause state, we would want to go back to ramp state here.
So you need to remember your previous state in a new variable.

11:08 pm

Sean Nash

We do not need to set result here since it is already set to run state at top of this function.

We do not need to set result here since it is already set to run state at top of this function.

11:07 pm

Sean Nash

Remove TODO comment since you have done it.

Remove TODO comment since you have done it.

11:06 pm

Sean Nash

Still need to close valves.

Still need to close valves.

11:06 pm

Sean Nash

Use getTargetBloodFlowRate() instead.

Use getTargetBloodFlowRate() instead.

11:05 pm

Sean Nash

Remove TODO comment since you have done it.

Remove TODO comment since you have done it.

11:04 pm

Sean Nash

Why assign requested rate to ramp rate? Why not just use requested rate in the set function below?

Why assign requested rate to ramp rate? Why not just use requested rate in the set function below?

11:03 pm

Sean Nash

I think we actually should be getting commanded blood flow rate here (getTargetBloodFlowRate) instead of measured rate.

I think we actually should be getting commanded blood flow rate here (getTargetBloodFlowRate) instead of measured rate.

10:58 pm

Sean Nash

Let's go ahead and do this.

Let's go ahead and do this.

10:57 pm

Sean Nash

Let's add a TODO comment so we don't forget to uncomment this when syringe pump is implemented.

Let's add a TODO comment so we don't forget to uncomment this when syringe pump is implemented.

10:55 pm

Sean Nash

Should be only one blank line here.

Should be only one blank line here.

10:55 pm

Sean Nash

I would expect there to be a change to Messaging.c (to add association between msg ID to this new function to the look-up table), but I don't see it.

I would expect there to be a change to Messaging.c (to add association between msg ID to this new function to the look-up table), but I don't see it.

10:52 pm

Sean Nash

I don't like controller bypassing driver to get to fpga access functions. There should be a get function in the driver that we can call here instead.

I don't like controller bypassing driver to get to fpga access functions. There should be a get function in the driver that we can call here instead.

10:51 pm

Sean Nash

Add blank line after declarations.

Add blank line after declarations.

10:49 pm

Sean Nash

Why two leading zeroes?

Why two leading zeroes?

10:02 am

Vendor - TEL - Sameer Poyil

Ya agreed. for the time being keeping it , we can later remove it after feature development complete

Ya agreed. for the time being keeping it , we can later remove it after feature development complete

9:15 am

Vendor - TEL - Jashwant Gantyada

in other files when they use string and/or math headers they had specified a comment describing which functions are used from the headers.

in other files when they use string and/or math headers they had specified a comment describing which functions are used from the headers.