hdfirmware / firmware / App / Modes

ModeTreatment.c

Clone Tools

Stats

Commits this week: 0
Total Committers: 11
Last Commit: 19 Nov
Commits this week: 0

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
Thursday 10 Apr
3:02 pm

Dara Navaei

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

Options
    • -1
    • +1
    ./ModeTreatment.c
  2. … 19 more files in changeset.
3:01 pm

Dara Navaei

2:58 pm

Dara Navaei

Merged release/p1b/staging

Options
    • -20
    • +131
    ./ModeTreatment.c
  2. … 17 more files in changeset.
Wednesday 09 Apr
4:45 pm

Michael Garthwaite and Sean Nash

Tuesday 08 Apr
4:23 pm

Dara Navaei

DENBUG-298 updated the code from staging

Options
    • -20
    • +131
    ./ModeTreatment.c
  2. … 19 more files in changeset.
2:25 pm

Dara Navaei

HD-DENBUG-298_User Unable TO Deliver Saline P1B Staging
HD-DENBUG-298_User Unable TO Deliver Saline P1B Staging
Friday 04 Apr
3:01 pm

Dara Navaei

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

Options
    • -1
    • +1
    ./ModeTreatment.c
  2. … 19 more files in changeset.
2:57 pm

Dara Navaei

1:56 pm

Michael Garthwaite and Sean Nash

12:50 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

12:48 pm

Dara Navaei

DENBUG-302 addressed the code review comment

Options
    • -1
    • +2
    ./ModeTreatment.c
12:47 pm

Dara Navaei

Done

Done

12:37 pm

Sean Nash

Bad practice to have two calls to same function here. Possible that 2nd call will return different result. Should call function once to assign value to a local var prior to if statement and then re...

Bad practice to have two calls to same function here. Possible that 2nd call will return different result.
Should call function once to assign value to a local var prior to if statement and then reference the local var in the if statement.

12:26 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:54 am

Dara Navaei

Yes. I think we should have these events to information.

Yes. I think we should have these events to information.

10:40 am

Dara Navaei

DENBUG-298 updated the VectorCAST test cases

Options
    • -1
    • +2
    ./ModeTreatment.c
Thursday 03 Apr
9:00 am

Sean Nash

Are these permanent?

Are these permanent?

Wednesday 02 Apr
5:30 pm

Dara Navaei

DENBUG-320 cleaned up the code

Options
    • -11
    • +8
    ./ModeTreatment.c
3:21 pm

Dara Navaei

DENBUG-302 updated the confirmation function to receive the 4 payloads to be sent up

Options
    • -1
    • +2
    ./ModeTreatment.c
  2. … 4 more files in changeset.
Tuesday 01 Apr
4:57 pm

Dara Navaei

DENBUG-302 added a timeout to the UF rate confirmation screen

Options
    • -2
    • +20
    ./ModeTreatment.c
8:34 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Monday 31 Mar
9:05 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Friday 28 Mar
4:45 pm

Dara Navaei

DENBUG-302 changed the UF rate that is reported to UI in mL/min

Options
    • -1
    • +1
    ./ModeTreatment.c
1:17 pm

Sean Nash

Add comment explaining what we're doing here.

Add comment explaining what we're doing here.

1:17 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

1:15 pm

Dara Navaei

Moved it to OperationModes

Moved it to OperationModes

1:14 pm

Dara Navaei

DENBUG-302 updated the code to check for the UF rate confirmation in operation mode. DEN-17237 changed the code to re-trigger the placeholder cleaning mode alarms if they got cleared

Options
    • -22
    • +29
    ./ModeTreatment.c
  2. … 3 more files in changeset.
11:00 am

Sean Nash

For other confirmations, an executive (e.g. alarm mgmt monitor or standby mode exec) is responsible for checking confirmation status and we don't call specific handlers from here. Why can't we foll...

For other confirmations, an executive (e.g. alarm mgmt monitor or standby mode exec) is responsible for checking confirmation status and we don't call specific handlers from here. Why can't we follow that design pattern for this confirmation?

10:25 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Thursday 27 Mar
6:01 pm

Dara Navaei

DENBUG-302 continued with the changes for the UF rate screen support

Options
    • -55
    • +51
    ./ModeTreatment.c
  2. … 9 more files in changeset.