hdfirmware / firmware / App

Services

Clone Tools

Stats

Commits this week: 0
Total Committers: 16
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 seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Monday 12 Apr 2021
10:09 am

Peman Montazemi <pmontazemi@diality.com>

DEN-7395: Addressed missing breaks in SystemComm.c

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

pmontazemi  (deleted user)

Good catch, addressed.

Good catch, addressed.

10:08 am

pmontazemi  (deleted user)

Good catch, addressed.

Good catch, addressed.

10:08 am

pmontazemi  (deleted user)

Good catch, addressed.

Good catch, addressed.

10:04 am

pmontazemi  (deleted user)

Addressed.

Addressed.

10:02 am

pmontazemi  (deleted user)

Addressed.

Addressed.

Sunday 11 Apr 2021
8:57 pm

Dara Navaei

Maybe I do not quite understand the architecture, but is there any reason that your are returning the state of the blood leak detector state machine?

Maybe I do not quite understand the architecture, but is there any reason that your are returning the state of the blood leak detector state machine?

8:54 pm

Dara Navaei

I suppose this is not the monitor function and it is the exec function?

I suppose this is not the monitor function and it is the exec function?

8:48 pm

Dara Navaei

I think it is a good practice to reset all the important variables that are defined as static in the .c file.

I think it is a good practice to reset all the important variables that are defined as static in the .c file.

8:41 pm

Dara Navaei

The states of the state machine should be private. This enum should be transferred to the .c file.

The states of the state machine should be private. This enum should be transferred to the .c file.

8:31 pm

Dara Navaei

A break is needed here.

A break is needed here.

8:30 pm

Dara Navaei

A break is needed here.

A break is needed here.

8:30 pm

Dara Navaei

A break is needed here.

A break is needed here.

4:45 pm

Sean Nash

I imagine this state should look for self-test to pass (indicate a blood leak) within some time period and then we would tell FPGA to put sensor in normal state and set your state machine to normal...

I imagine this state should look for self-test to pass (indicate a blood leak) within some time period and then we would tell FPGA to put sensor in normal state and set your state machine to normal state. Fault if times out.

4:44 pm

Sean Nash

I imagine this state would need to look for zero to complete within some time period so we can tell FPGA to put sensor in self-test mode and set your state machine to self-test state. Fault if time...

I imagine this state would need to look for zero to complete within some time period so we can tell FPGA to put sensor in self-test mode and set your state machine to self-test state. Fault if times out.

4:43 pm

Sean Nash

Where did zeroBloodLeak() and selfTestBloodLeak() functions go? You have prototypes for them in .h and I think you still need at least one of them. zeroBloodLeak would be called by Pre-Treatment Mo...

Where did zeroBloodLeak() and selfTestBloodLeak() functions go? You have prototypes for them in .h and I think you still need at least one of them. zeroBloodLeak would be called by Pre-Treatment Mode and would tell FPGA to put sensor in zero state and would also put your state machine in zero state.

4:32 pm

Sean Nash

initBloodLeak() was already called from main in sys_main.c. I don't think we need to keep calling it here. How do we get out of INIT state?

initBloodLeak() was already called from main in sys_main.c. I don't think we need to keep calling it here.
How do we get out of INIT state?

Saturday 10 Apr 2021
11:42 pm

pmontazemi  (deleted user)

Addressed.

Addressed.

11:41 pm

Peman Montazemi <pmontazemi@diality.com>

DEN-7395: Implemented state machine architecture for Blood Leak Detector driver

Options
    • -0
    • +1
    ./AlarmMgmt.h
  2. … 1 more file in changeset.
8:54 pm

pmontazemi  (deleted user)

Addressed.

Addressed.

Friday 09 Apr 2021
8:20 pm

pmontazemi  (deleted user)

Addressed.

Addressed.

8:20 pm

pmontazemi  (deleted user)

Addressed.

Addressed.

8:09 pm

Peman Montazemi <pmontazemi@diality.com>

DEN-7395: Reverted lumping of override functions for Blood Leak Detector status and state

Options
    • -2
    • +5
    ./SystemComm.c
    • -5
    • +37
    ./SystemCommMessages.c
    • -2
    • +5
    ./SystemCommMessages.h
  4. … 2 more files in changeset.
12:25 pm

Sean Nash

Merge branch 'DEN-7605-hd_dg_dev-self-tests' into develop

Options
    • -0
    • +1
    ./SystemCommMessages.c
  2. … 1 more file in changeset.
12:25 pm

Sean Nash

DEN-7605: Added no minimize flag to alarm status message to UI.

Options
    • -1
    • +9
    ./AlarmMgmt.c
    • -0
    • +1
    ./AlarmMgmt.h
    • -0
    • +1
    ./SystemCommMessages.c
  4. … 1 more file in changeset.
10:52 am

Sean Nash

There is only one state in this override message. It can be status or state (depending on message ID), but not both. Whichever it is, you are passing it to a function twice as if it is two differen...

There is only one state in this override message. It can be status or state (depending on message ID), but not both. Whichever it is, you are passing it to a function twice as if it is two different things. You need to revert back to having two override messages and two message handlers - one for status and one for state.

10:45 am

Sean Nash

So you combined the published data into one message in the publish function above which is correct. But these overrides must be separate. You should only override one thing at a time, so overrides ...

So you combined the published data into one message in the publish function above which is correct. But these overrides must be separate. You should only override one thing at a time, so overrides were ok as it was before.

10:31 am

qnguyen  (deleted user)

Need to add call to this function from task priority.

Need to add call to this function from task priority.

Thursday 08 Apr 2021
5:49 pm

Sean Nash

DEN-7605: Fixes per verification testing.

Options
    • -1
    • +2
    ./AlarmMgmt.c
  2. … 5 more files in changeset.
3:17 pm

Peman Montazemi <pmontazemi@diality.com>

DEN-7395: Removed Detector from names of functions

Options
    • -4
    • +4
    ./SystemComm.c
    • -4
    • +4
    ./SystemCommMessages.c
    • -4
    • +4
    ./SystemCommMessages.h