hdfirmware / firmware / App / Services

SystemComm.c

Clone Tools

Stats

Commits this week: 0
Total Committers: 15
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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 17 Feb 2022
11:23 am

Hung Nguyen <hnguyen@diality.com>

Merge branch 'DEN-11250-hd-syringe-pump-updates' into staging

Options
    • -0
    • +3
    ./SystemComm.c
10:37 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:34 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:34 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:34 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:33 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:33 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:33 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:33 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:33 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:32 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:32 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:32 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:30 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Wednesday 16 Feb 2022
1:55 pm

Michael Garthwaite

Please see comment below regarding the changes in common from this code review.

Please see comment below regarding the changes in common from this code review.

1:54 pm

Michael Garthwaite

Dara and I have found out that this code review for common is misconfigured. The source and destination branches are misconfigured. The branch to review should have been from DEN-11980_sw_dev_sprin...

Dara and I have found out that this code review for common is misconfigured. The source and destination branches are misconfigured. The branch to review should have been from DEN-11980_sw_dev_sprint_64 and the "branch from" should have been from staging. This code review has staging as the branch to review with master as the "branch review."

12:39 pm

Hung Nguyen <hnguyen@diality.com>

DEN-11250 Installed case MSG_ID_HD_FAN_RPM_ALARM_START_TIME_OFFSET_OVERRIDE per peer review comments

Options
    • -0
    • +3
    ./SystemComm.c
12:34 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

12:34 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

12:33 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

12:32 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

12:32 pm

Michael Garthwaite

Renamed to testSendCurrentTreatmentParameters(). thanks!

Renamed to testSendCurrentTreatmentParameters(). thanks!

12:31 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

12:31 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

12:31 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

12:31 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:49 am

hnguyen  (deleted user)

Reinstalled the case statement back to where it was. case MSG_ID_HD_FAN_RPM_ALARM_START_TIME_OFFSET_OVERRIDE: handleTestFansRPMAlarmStartTimeOffsetOverrideRequest( message );

Reinstalled the case statement back to where it was.
case MSG_ID_HD_FAN_RPM_ALARM_START_TIME_OFFSET_OVERRIDE:
handleTestFansRPMAlarmStartTimeOffsetOverrideRequest( message );

10:13 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:09 am

Sean Nash

Function name is misleading. Sounds like Dialin is sending us new parameters, but it is only requesting current params be sent. I would change to something like testSendCurrentTreatmentParameters().

Function name is misleading. Sounds like Dialin is sending us new parameters, but it is only requesting current params be sent. I would change to something like testSendCurrentTreatmentParameters().

10:06 am

Sean Nash

Check that Dialin has logged in before acting on a Dialin request (TRUE == isTestingActivated()).

Check that Dialin has logged in before acting on a Dialin request (TRUE == isTestingActivated()).