common

Clone Tools

Stats

Commits this week: 0
Total Committers: 18
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
Wednesday 19 May 2021
4:57 pm

Sean Nash

Change PERIOD to PERIODIC?

Change PERIOD to PERIODIC?

4:55 pm

Sean Nash

How long does this take? We may need to break this loop into smaller pieces and have ModeInitPOST call this function multiple times so that general task does not take too long (> 40ms or so). Other...

How long does this take? We may need to break this loop into smaller pieces and have ModeInitPOST call this function multiple times so that general task does not take too long (> 40ms or so). Otherwise, watchdog will not get pet within its ~50ms window and expire - and also other general and background operations will be blocked until this test is completed.

4:49 pm

Sean Nash

This enum is getting big. Should we put it in its own .h file (AlarmMgmtSWFaults.h) with same doxygen group name (AlarmManagement) that would be included by this header?

This enum is getting big. Should we put it in its own .h file (AlarmMgmtSWFaults.h) with same doxygen group name (AlarmManagement) that would be included by this header?

4:42 pm

Sean Nash

For UF, need to accumulate volume deltas or remember where 30 min volume started and then subtract from current volume when 30 minutes is up. Then convert to rate by dividing volume by 30 min.

For UF, need to accumulate volume deltas or remember where 30 min volume started and then subtract from current volume when 30 minutes is up. Then convert to rate by dividing volume by 30 min.

4:38 pm

qnguyen  (deleted user)

Done.

Done.

4:38 pm

qnguyen  (deleted user)

Done.

Done.

4:35 pm

Sean Nash

Neither of these if statements should be executed if treatment is paused (i.e. treatment time is counting down, i.e. we're in dialysis sub-mode of treatment mode and not delivering a saline bolus).

Neither of these if statements should be executed if treatment is paused (i.e. treatment time is counting down, i.e. we're in dialysis sub-mode of treatment mode and not delivering a saline bolus).

4:29 pm

Sean Nash

Let UI timestamp

Let UI timestamp

4:09 pm

qnguyen  (deleted user)

The UI will provide this. Firmware will not have this value.

The UI will provide this. Firmware will not have this value.

3:46 pm

Sean Nash

Need to add a get function for private variable totalSalineVolumeDelivered in Dialysis module.

Need to add a get function for private variable totalSalineVolumeDelivered in Dialysis module.

3:44 pm

Sean Nash

This should be available from syringe pump driver - getSyringePumpVolumeDelivered().

This should be available from syringe pump driver - getSyringePumpVolumeDelivered().

3:41 pm

Sean Nash

It doesn't look like this function sends anything to UI - just setting an epoch. Does f/w need to log treatment start/end time or can UI do that from its own clock.

It doesn't look like this function sends anything to UI - just setting an epoch. Does f/w need to log treatment start/end time or can UI do that from its own clock.

3:39 pm

Sean Nash

Do we not need concentration?

Do we not need concentration?

Tuesday 18 May 2021
10:42 am

qnguyen  (deleted user)

This will be addressed in DEN-8103.

This will be addressed in DEN-8103.

10:40 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:40 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:39 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:09 am

qnguyen  (deleted user)

HD-DEN-8103_DG HD Dev Alarm Design
HD-DEN-8103_DG HD Dev Alarm Design
Tuesday 11 May 2021
2:54 pm

Sean Nash

Agreed. This is for me to add a check for this error. Will remove comment when I'm done.

Agreed. This is for me to add a check for this error. Will remove comment when I'm done.

2:54 pm

Sean Nash

Is this still an issue?

Is this still an issue?

2:53 pm

Sean Nash

Agreed. This is for me to add a check for this error. I'll remove comment when I'm done.

Agreed. This is for me to add a check for this error. I'll remove comment when I'm done.

2:53 pm

Sean Nash

Is this still an issue?

Is this still an issue?

Friday 07 May 2021
12:07 am

qnguyen  (deleted user)

Need to remove this comment.

Need to remove this comment.

12:07 am

qnguyen  (deleted user)

Need to remove this comment.

Need to remove this comment.

12:06 am

qnguyen  (deleted user)

Same as comment from DialinFlow.c

Same as comment from DialinFlow.c

12:06 am

qnguyen  (deleted user)

We can't have 3 calls to isPersistentAlarmTriggered() on same alarm. If the first call starts the timer and the second or third call is good, it will reset the timer to 0 again. Suggest creating a ...

We can't have 3 calls to isPersistentAlarmTriggered() on same alarm. If the first call starts the timer and the second or third call is good, it will reset the timer to 0 again.
Suggest creating a BOOL variable to have all three conditions or with each other and call isPersistentAlarmTriggered() only once.

Thursday 06 May 2021
10:27 am

Sean Nash

initBloodPrime() is called every time we transition to blood prime sub-mode.

initBloodPrime() is called every time we transition to blood prime sub-mode.

Wednesday 05 May 2021
7:57 pm

qnguyen  (deleted user)

Blood prime last motor count does not seem to get updated except in initBloodPrime(). Should it be updated continuously somewhere?

Blood prime last motor count does not seem to get updated except in initBloodPrime(). Should it be updated continuously somewhere?

Tuesday 04 May 2021
10:38 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:37 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.