hdfirmware / firmware / App

Modes

Clone Tools

Stats

Commits this week: 0
Total Committers: 15
Last Commit: 19 Nov 25
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 30 Jun 2022
2:37 pm

Darren Cox <darren_m@cox.net>

DEN-12847 code review comments update.

Options
    • -73
    • +45
    ./Dialysis.c
2:18 pm

dcox  (deleted user)

Per conversation with Sean: It is a DG Fault generated that the HD is identifying.

Per conversation with Sean: It is a DG Fault generated that the HD is identifying.

2:13 pm

dcox  (deleted user)

Leave as is per conversation w Sean. Reported by HD, but actually a DG generated problem.

Leave as is per conversation w Sean. Reported by HD, but actually a DG generated problem.

12:45 pm

wbracken  (deleted user)

General comment: Should floating point values be of the format N.NF? What is the coding standard? My experience has been that it's best to always specify the "F".

General comment: Should floating point values be of the format N.NF? What is the coding standard? My experience has been that it's best to always specify the "F".

10:09 am

Dara Navaei

10:02 am

Sean Nash

This has the rank of a fault and I think it should be an HD fault - so set the T/F properties same as the alarm above (FPGA clock speed check).

This has the rank of a fault and I think it should be an HD fault - so set the T/F properties same as the alarm above (FPGA clock speed check).

9:44 am

Sean Nash

Maybe change param name to final (i.e. final or start).

Maybe change param name to final (i.e. final or start).

9:42 am

Sean Nash

I think we can handle both reservoirs with same code (no need for else). Probably just need to add a macro somewhere that converts a reservoir ID and primary/backup selection into a loadcell ID.

I think we can handle both reservoirs with same code (no need for else). Probably just need to add a macro somewhere that converts a reservoir ID and primary/backup selection into a loadcell ID.

9:39 am

Sean Nash

I think this whole condition should be removed. If we are taking baseline reading for a reservoir, it is not time to do the drift check yet. Just need to save the readings here.

I think this whole condition should be removed. If we are taking baseline reading for a reservoir, it is not time to do the drift check yet. Just need to save the readings here.

9:37 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

9:37 am

Sean Nash

Remove blank line.

Remove blank line.

9:37 am

Sean Nash

I think this alarm should be an HD fault for now (no recovery).

I think this alarm should be an HD fault for now (no recovery).

9:34 am

Sean Nash

Do an explicit condition here. i.e. if ( TRUE == reservoirFull )

Do an explicit condition here. i.e. if ( TRUE == reservoirFull )

9:27 am

Sean Nash

It's not so much empty and full - neither is empty - it's more like start and final weights. Consider renaming these. Also, consider moving these declarations to top of module. Nothing wrong with t...

It's not so much empty and full - neither is empty - it's more like start and final weights. Consider renaming these.
Also, consider moving these declarations to top of module. Nothing wrong with them being here, but the style for this code base has been to declare everything at top of module.

9:19 am

Sean Nash

Use FALSE, not false.

Use FALSE, not false.

9:18 am

Sean Nash

Use TRUE (not true).

Use TRUE (not true).

9:13 am

Sean Nash

Is fabs() appropriate here?

Is fabs() appropriate here?

9:12 am

Sean Nash

Is fabs() appropriate here?

Is fabs() appropriate here?

Tuesday 28 Jun 2022
3:50 pm

Darren Cox <darren_m@cox.net>

DEN-12847 Code review comment updates.

Options
    • -4
    • +1
    ./Dialysis.c
2:31 pm

Darren Cox <darren_m@cox.net>

DEN12847 Code review comments update 1.

Options
    • -1
    • +0
    ./Dialysis.c
  2. … 1 more file in changeset.
10:09 am

Michael Garthwaite

This #define should be at the top of the file.

This #define should be at the top of the file.

Monday 27 Jun 2022
4:14 pm

Bill Bracken <bill@criticalsoftwaresolutions.com>

Merge remote-tracking branch 'origin/DEN-13141-miscellaneous_hd_issues_bb' into develop

Options
    • -6
    • +0
    ./SelfTests.c
  2. … 1 more file in changeset.
4:12 pm

Bill Bracken <bill@criticalsoftwaresolutions.com>

DEN-12974 DEN-13067 Return execUITest to original state.

Options
    • -0
    • +2
    ./ModeInitPOST.c
Friday 24 Jun 2022
11:39 am

Dara Navaei

HD-DEN-12847_SW Dev Sprint 71 Darren
HD-DEN-12847_SW Dev Sprint 71 Darren
Thursday 23 Jun 2022
2:17 pm

Bill Bracken <bill@criticalsoftwaresolutions.com>

DEN-13141, DEN-13067, DEN013116 Correct POST messaging for UI. Continue to use #define to remove fault testing. Issue is a synchronization issue between HD and UI. Remove check for generating ALARM_ID_PRE_TREATMENT_DRY_PRESSURE_TEST_FAILURE which could never occur.

Options
    • -1
    • +4
    ./ModeInitPOST.c
    • -8
    • +0
    ./SelfTests.c
12:46 pm

Dara Navaei

Merged DEN-12931

Options
    • -7
    • +10
    ./ModePreTreat.c
  2. … 5 more files in changeset.
Wednesday 22 Jun 2022
4:52 pm

Bill Bracken <bill@criticalsoftwaresolutions.com>

DEN-13141, DEN-13067 Correct POST messaging for UI.

Options
    • -4
    • +1
    ./ModeInitPOST.c
    • -1
    • +3
    ./SelfTests.c
  3. … 1 more file in changeset.
Monday 20 Jun 2022
9:38 am

Dara Navaei

DEN-12931 merged staging and resolved the conflicts

Options
    • -7
    • +10
    ./ModePreTreat.c
  2. … 3 more files in changeset.
Friday 17 Jun 2022
1:47 pm

Darren Cox <darren_m@cox.net>

DEN-12847 Remove Debug output. Change drift limit.

Options
    • -5
    • +0
    ./Dialysis.c
  2. … 1 more file in changeset.
Thursday 16 Jun 2022
5:34 pm

Dara Navaei