common

Clone Tools

Stats

Commits this week: 0
Total Committers: 18
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
Friday 05 Feb 2021
2:38 pm

Sean Nash

DEN-6372: Added new Treatment End sub-mode module.

Options
    • -0
    • +2
    /AlarmDefs.h
    • -2
    • +31
    /HDDefs.h
9:24 am

Dara Navaei

Thursday 04 Feb 2021
9:51 am

Quang Nguyen <qnguyen@diality.com>

Merge branch 'Sprint37' into develop

Options
9:51 am

Quang Nguyen <qnguyen@diality.com>

DEN-6402: Added new alarm for no cartridge loaded

Options
    • -0
    • +2
    /AlarmDefs.h
    • -2
    • +2
    /HDDefs.h
8:59 am

qnguyen  (deleted user)

When we run blood pump in forward direction, we should see negative on arterial and positive on venous. This limit here is to stop blood pump even if the time to run the blood has not been reached.

When we run blood pump in forward direction, we should see negative on arterial and positive on venous.
This limit here is to stop blood pump even if the time to run the blood has not been reached.

Wednesday 03 Feb 2021
6:14 pm

qnguyen  (deleted user)

Done. Also moved the function to FPGA module.

Done. Also moved the function to FPGA module.

6:14 pm

qnguyen  (deleted user)

Done.

Done.

6:10 pm

qnguyen  (deleted user)

Done.

Done.

6:10 pm

qnguyen  (deleted user)

Reworded.

Reworded.

5:22 pm

qnguyen  (deleted user)

Renamed.

Renamed.

5:17 pm

qnguyen  (deleted user)

If the sensor is bad, we will go to fault. In this case, the calibration does not matter anymore.

If the sensor is bad, we will go to fault. In this case, the calibration does not matter anymore.

5:16 pm

qnguyen  (deleted user)

Agree. All multi-lines state have been moved to a static function.

Agree. All multi-lines state have been moved to a static function.

4:50 pm

qnguyen  (deleted user)

Done.

Done.

4:50 pm

qnguyen  (deleted user)

Done. It is a tolerance on pressure difference after we return back to normal state from pressure self-tests.

Done. It is a tolerance on pressure difference after we return back to normal state from pressure self-tests.

4:48 pm

qnguyen  (deleted user)

Done. This is not a self-test for blood pump occlusion sensor (OB). This is for PBa and PBo pressure sensors.

Done. This is not a self-test for blood pump occlusion sensor (OB). This is for PBa and PBo pressure sensors.

4:47 pm

qnguyen  (deleted user)

Done.

Done.

4:47 pm

qnguyen  (deleted user)

Done.

Done.

9:45 am

Sean Nash

Merge branch 'Sprint37' into develop

Options
9:35 am

Sean Nash

Fixed comment typo.

Options
    • -1
    • +1
    /HDDefs.h
Tuesday 02 Feb 2021
3:21 pm

Sean Nash

Fixed comment alignment.

Options
    • -1
    • +1
    /AlarmDefs.h
10:37 am

Dara Navaei

DG-DEN-5980_Non Volatile Data Management (2 of 2)
DG-DEN-5980_Non Volatile Data Management (2 of 2)
Monday 01 Feb 2021
11:56 pm

Sean Nash

Fair point.

Fair point.

11:52 pm

Sean Nash

Consider returning a OPN_CLS_STATE_T instead of BOOL for readability.

Consider returning a OPN_CLS_STATE_T instead of BOOL for readability.

11:50 pm

Sean Nash

What is this state? Where did it get set? Why aren't we going back to recirc state?

What is this state? Where did it get set? Why aren't we going back to recirc state?

11:45 pm

Sean Nash

Can this wait for door close be a state within the self-test no cartridge sub-mode? Also need to wait for user to open the door at the end of self-test no cartridge sub-mode or beginning of cartrid...

Can this wait for door close be a state within the self-test no cartridge sub-mode? Also need to wait for user to open the door at the end of self-test no cartridge sub-mode or beginning of cartridge install sub-mode and then close the door again at end of cartridge install sub-mode. I see that some of these sub-modes don't have states, but probably should or will eventually. The 3 self-test sub-modes can have their own state machines in SelfTests.c. I think Recirc and Cartridge Install sub-modes may need their own .c files (PreTreatmentRecirc.c and DisposableInstall.c?) with state machines of their own as well.

11:40 pm

Sean Nash

Move execPrime() to end - only if no door open and no alarm stop signal.

Move execPrime() to end - only if no door open and no alarm stop signal.

11:36 pm

Sean Nash

Check for door open before running self-tests. Really, self-test execution should go last and only if no door open and no stop signal.

Check for door open before running self-tests. Really, self-test execution should go last and only if no door open and no stop signal.

11:35 pm

qnguyen  (deleted user)

For no cartridge self-tests, all failures result in a fault because they are essentially POST tests.

For no cartridge self-tests, all failures result in a fault because they are essentially POST tests.

11:33 pm

Sean Nash

Are these the only two alarms possible in this state? Consider using the alarm stop signal here instead of checking specific alarm status.

Are these the only two alarms possible in this state? Consider using the alarm stop signal here instead of checking specific alarm status.

11:29 pm

Sean Nash

And if we fail self-tests? Where do we go? Are all failures a fault?

And if we fail self-tests? Where do we go? Are all failures a fault?