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
Monday 22 Mar 2021
5:30 pm

pmontazemi  (deleted user)

Either add TODO or delete commented line.

Either add TODO or delete commented line.

5:24 pm

pmontazemi  (deleted user)

Not better to use this instead? bloodPumpMotorEdgeCount += (U16)(delta);

Not better to use this instead? bloodPumpMotorEdgeCount += (U16)(delta);

5:17 pm

Sean Nash

Can we log 2 data with alarms (set alarm and a data value when detected, log with alarm here)?

Can we log 2 data with alarms (set alarm and a data value when detected, log with alarm here)?

5:12 pm

Sean Nash

Is this the new "coldest spot" sensor?

Is this the new "coldest spot" sensor?

5:11 pm

Sean Nash

Can drain time out in this state? Why no check?

Can drain time out in this state? Why no check?

5:08 pm

Sean Nash

What is this state doing?

What is this state doing?

5:08 pm

Sean Nash

Are these the only two faults that can happen in R1 to R2 state?

Are these the only two faults that can happen in R1 to R2 state?

4:57 pm

Sean Nash

I prefer to reverse the order of these to change accidental = instead of ==.

I prefer to reverse the order of these to change accidental = instead of ==.

4:53 pm

Sean Nash

Check boolean explicitly.

Check boolean explicitly.

4:49 pm

Sean Nash

Check these booleans explicitly.

Check these booleans explicitly.

4:48 pm

Sean Nash

Set these booleans to TRUE or FALSE explicitly.

Set these booleans to TRUE or FALSE explicitly.

4:47 pm

Sean Nash

Is Thd same as TRo? Does it belong in this group?

Is Thd same as TRo? Does it belong in this group?

4:44 pm

qnguyen  (deleted user)

Removed. It seems nothing much we can do. The door switch is simple and just a GPIO read from FPGA.

Removed. It seems nothing much we can do. The door switch is simple and just a GPIO read from FPGA.

4:43 pm

qnguyen  (deleted user)

Straw door needs to be open for concentrate jugs' connection. Other than that, I am not aware of any flow path that requires closed door. Added TODO.

Straw door needs to be open for concentrate jugs' connection. Other than that, I am not aware of any flow path that requires closed door.
Added TODO.

4:42 pm

Sean Nash

This comment needs a rephrase. I don't think range has anything to do with it. This is a retry counter check.

This comment needs a rephrase. I don't think range has anything to do with it. This is a retry counter check.

4:41 pm

qnguyen  (deleted user)

Done.

Done.

4:41 pm

qnguyen  (deleted user)

Combined air trap and bubble detectors into one used cartridge check.

Combined air trap and bubble detectors into one used cartridge check.

4:41 pm

qnguyen  (deleted user)

Combined air trap and bubble detectors into one used cartridge check.

Combined air trap and bubble detectors into one used cartridge check.

4:38 pm

Sean Nash

Recommend being much more descriptive of these states. What exactly is the state trying to accomplish.

Recommend being much more descriptive of these states. What exactly is the state trying to accomplish.

4:33 pm

Sean Nash

Why no actuators set for draining R2 here? I see VRd being set a few lines down. Maybe move this comment to go with the valve change.

Why no actuators set for draining R2 here? I see VRd being set a few lines down. Maybe move this comment to go with the valve change.

4:29 pm

qnguyen  (deleted user)

Checked with Nick. The DG does not expose motors to the user. The straw door switch should be open in Fill Mode to allow access to straw connection. The straw door switch should be closed in modes ...

Checked with Nick. The DG does not expose motors to the user.
The straw door switch should be open in Fill Mode to allow access to straw connection.
The straw door switch should be closed in modes such as (heat disinfection, chem disinfection) to prevent user from exposure to hot water or chemical.

Added TODO.

4:16 pm

Sean Nash

I prefer boolean conditions be explicit (e.g. TRUE == isThisLastDrain). This way, if value gets corrupted to a garbage value (like 5) it will be same as FALSE.

I prefer boolean conditions be explicit (e.g. TRUE == isThisLastDrain). This way, if value gets corrupted to a garbage value (like 5) it will be same as FALSE.

4:12 pm

Sean Nash

Should actuators be reset here too?

Should actuators be reset here too?

4:10 pm

Sean Nash

This doesn't look like a complete list of possible status. What are these used for?

This doesn't look like a complete list of possible status. What are these used for?

4:08 pm

Sean Nash

I thought hot and cold paths got merged into one?

I thought hot and cold paths got merged into one?

4:07 pm

Sean Nash

When is status unknown? And shouldn't there be a partially full status?

When is status unknown? And shouldn't there be a partially full status?

4:03 pm

Sean Nash

Align #define values on left.

Align #define values on left.

3:49 pm

Sean Nash

This looks tight. Will we pass? Do we need a bigger filter for temp sensors?

This looks tight. Will we pass? Do we need a bigger filter for temp sensors?

3:44 pm

qnguyen  (deleted user)

Wrong valve state. Fixed.

Wrong valve state. Fixed.

3:44 pm

qnguyen  (deleted user)

Fixed.

Fixed.