hdfirmware / firmware

App

Clone Tools

Stats

Commits this week: 0
Total Committers: 17
Last Commit: 10 Jun
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 03 Mar 2022
12:51 pm

Dara Navaei

DEN-11750 addressed the code review comments

Options
    • -1
    • +1
    ./Controllers/SyringePump.c
    • -1
    • +2
    ./Controllers/Temperatures.c
    • -3
    • +0
    ./Modes/ModeInitPOST.c
    • -98
    • +102
    ./Services/SystemCommMessages.c
    • -1
    • +0
    ./Services/SystemCommMessages.h
12:50 pm

Dara Navaei

Done.

Done.

12:48 pm

Dara Navaei

Done.

Done.

Wednesday 02 Mar 2022
8:49 pm

Dara Navaei

Done.

Done.

8:48 pm

Dara Navaei

No, this should be removed. If this is in an else, in a release build it is not going to compile sine the if will be ignored.

No, this should be removed. If this is in an else, in a release build it is not going to compile sine the if will be ignored.

5:20 pm

Sean Nash

These appear to be Dialin message functions. They should be down in the Dialin section below the TEST SUPPORT FUNCTIONS banner.

These appear to be Dialin message functions. They should be down in the Dialin section below the TEST SUPPORT FUNCTIONS banner.

5:10 pm

Sean Nash

Check payload length is at least 12.

Check payload length is at least 12.

5:09 pm

Sean Nash

Remove extra blank line.

Remove extra blank line.

5:02 pm

Sean Nash

This can be uninitialized. It is initialized in the init function below.

This can be uninitialized. It is initialized in the init function below.

4:43 pm

Sean Nash

Should this be in an else below?

Should this be in an else below?

4:36 pm

Sean Nash

There should be 2 blank lines above and below the TEST SUPPORT FUNCTIONS banner below.

There should be 2 blank lines above and below the TEST SUPPORT FUNCTIONS banner below.

4:34 pm

Dara Navaei

Done.

Done.

4:34 pm

Sean Nash

Can leave uninitialized here. It is initialized in init function below.

Can leave uninitialized here. It is initialized in init function below.

4:34 pm

Dara Navaei

Done.

Done.

4:31 pm

Dara Navaei

It has been removed.

It has been removed.

4:27 pm

Sean Nash

Add TODO to remove later.

Add TODO to remove later.

4:26 pm

Sean Nash

Add TODO to remove.

Add TODO to remove.

4:24 pm

Sean Nash

Add one more blank line here.

Add one more blank line here.

4:05 pm

Dara Navaei

Yes, there might be other release issues. We have not built release in a long time but we can resolve them once we decided to have a release build.

Yes, there might be other release issues. We have not built release in a long time but we can resolve them once we decided to have a release build.

3:56 pm

Sean Nash

It doesn't appear to build for me.

It doesn't appear to build for me.

3:55 pm

Sean Nash

I reviewed these - fixed any differences due to merge. There is one more threshold change pending that I'm aware of. I will make in next branch.

I reviewed these - fixed any differences due to merge. There is one more threshold change pending that I'm aware of. I will make in next branch.

3:49 pm

Sean Nash

DEN-11750: Fixed merge issues.

Options
    • -4
    • +2
    ./Controllers/PresOccl.c
    • -8
    • +11
    ./Modes/SelfTests.c
3:47 pm

Darren Cox <darren_m@cox.net>

DEN-10974 Added new alarm for pump track latch open. Check Syringe installed with doors. Clear Door and Pump Track alarms in Switches driver.

Options
    • -0
    • +21
    ./Controllers/Switches.c
    • -3
    • +16
    ./Modes/ModePreTreat.c
3:04 pm

Dara Navaei

DEN-11750 addressed the code review comments

Options
    • -6
    • +7
    ./Controllers/Fans.c
2:27 pm

Dara Navaei

DEN-11750 addressed the code review comments

Options
    • -1
    • +1
    ./Controllers/BloodFlow.c
    • -4
    • +2
    ./Controllers/Fans.c
    • -2
    • +0
    ./Controllers/Fans.h
    • -3
    • +5
    ./Controllers/PresOccl.c
2:05 pm

Dara Navaei

Done.

Done.

1:59 pm

Dara Navaei

Yes, I have a TODO to finish them off in DEN-12224.

Yes, I have a TODO to finish them off in DEN-12224.

1:54 pm

Dara Navaei

Done.

Done.

1:52 pm

Dara Navaei

Done.

Done.

1:51 pm

Dara Navaei

Yes, I tried a release build and it compiled.

Yes, I tried a release build and it compiled.