hdfirmware

firmware

Clone Tools

Stats

Commits this week: 0
Total Committers: 17
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 02 Jun 2023
1:52 pm

jtaylor  (deleted user)

Done. Changed each instance to the negative form for consistency.

Done. Changed each instance to the negative form for consistency.

1:52 pm

jtaylor  (deleted user)

Done. Changed each instance to the negative form for consistency.

Done. Changed each instance to the negative form for consistency.

1:51 pm

jtaylor  (deleted user)

We appear to need a delay from AC's return to the first alarm check. We have natural times at CPLD, Alarm clear, and the end of the alarm block interval. The alarm block timer currently clears 10 s...

We appear to need a delay from AC's return to the first alarm check. We have natural times at CPLD, Alarm clear, and the end of the alarm block interval. The alarm block timer currently clears 10 seconds after CPLD; the alarm is clear at a random interval from milliseconds to minutes after CPLD.
I have modified the calls for consistent use of CPLD or of isACPowerLost.

1:49 pm

jtaylor  (deleted user)

Done. The countdown timer was renamed for clarity.

Done. The countdown timer was renamed for clarity.

1:49 pm

jtaylor  (deleted user)

Done.

Done.

1:48 pm

jtaylor  (deleted user)

Renamed for clarification.

Renamed for clarification.

1:46 pm

jtaylor  (deleted user)

Done. Changed each instance to the negative form for consistency.

Done. Changed each instance to the negative form for consistency.

12:48 pm

jtaylor  (deleted user)

Done. Changed each instance to the negative form for consistency.

Done. Changed each instance to the negative form for consistency.

12:10 pm

Michael Garthwaite

12:03 pm

jtaylor  (deleted user) and Sean Nash

11:48 am

Sean Nash

11:46 am

Michael Garthwaite

11:26 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

11:24 am

Michael Garthwaite

fixed. set to no sub state (0xFF) once we transition to pretx recirc

fixed. set to no sub state (0xFF) once we transition to pretx recirc

11:12 am

Sean Nash

Looks like this is only being set in wet self-tests substate of pre-tx prime. Once we leave wet self-tests, should this 4th level state be set back to zero or something?

Looks like this is only being set in wet self-tests substate of pre-tx prime. Once we leave wet self-tests, should this 4th level state be set back to zero or something?

Thursday 01 Jun 2023
8:22 pm

Sean Nash

Seems like we should be calling CPLD function here and in Fans, DialInFlow, and DGInterface too. Also recommend calling the isACPowerLost() function in just one place - maybe alarm mgmt exec to gi...

Seems like we should be calling CPLD function here and in Fans, DialInFlow, and DGInterface too.

Also recommend calling the isACPowerLost() function in just one place - maybe alarm mgmt exec to give you your 10 sec alarm block. Maybe rename the function to checkACPowerLost() with no return value and make it static (private).

8:17 pm

Sean Nash

It is a timer counter. Counts down to zero. And still need to address function header per Bill's first comment.

It is a timer counter. Counts down to zero.

And still need to address function header per Bill's first comment.

8:16 pm

Sean Nash

This seems unnecessary now with your new function doing this all the time. Consider removing.

This seems unnecessary now with your new function doing this all the time. Consider removing.

8:13 pm

Sean Nash

This is technically a timer counter based on task tick of 50ms.

This is technically a timer counter based on task tick of 50ms.

8:11 pm

Sean Nash

This was safer the other way. Why change?

This was safer the other way. Why change?

8:11 pm

Sean Nash

This was safer the other way. Why change?

This was safer the other way. Why change?

8:10 pm

Sean Nash

This was safer the other way. Why change?

This was safer the other way. Why change?

8:10 pm

Sean Nash

This was safer the other way. Why change?

This was safer the other way. Why change?

8:07 pm

Sean Nash

How is this different than checking CPLD? Condition is cleared when CPLD no longer sees power loss (w/ 150ms delay I guess). Seems simpler to use CPLD function instead of checking 2 alarms.

How is this different than checking CPLD? Condition is cleared when CPLD no longer sees power loss (w/ 150ms delay I guess). Seems simpler to use CPLD function instead of checking 2 alarms.

8:02 pm

Sean Nash

Remove extra blank line.

Remove extra blank line.

6:05 pm

Dara Navaei

6:05 pm

Dara Navaei

5:09 pm

Dara Navaei

HD-DEN-15317_Usability HD Door Open Alarm During Post Treatment
HD-DEN-15317_Usability HD Door Open Alarm During Post Treatment
4:33 pm

dcox  (deleted user)

4:24 pm

Sean Nash , jtaylor  (deleted user) and wbracken  (deleted user)