leahi-common

Clone Tools

Stats

Commits this week: 19
Total Committers: 17
Last Commit: 25 Nov 24
Commits this week: 19

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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Today
1 min ago

Sean Nash

I think this driver should be moved to FWCommon folder as it should work the same for both DD and TD firmware.

I think this driver should be moved to FWCommon folder as it should work the same for both DD and TD firmware.

2 mins ago

Sean Nash

Add a blank line before group end marker.

Add a blank line before group end marker.

3 mins ago

Sean Nash

No load cells in Leahi.

No load cells in Leahi.

5 mins ago

Sean Nash

I think this enum belongs to NVRecords (these are records, not jobs).

I think this enum belongs to NVRecords (these are records, not jobs).

6 mins ago

Sean Nash

I think this enum belongs to the Q (these are Q jobs), not in driver.

I think this enum belongs to the Q (these are Q jobs), not in driver.

8 mins ago

Sean Nash

Do these need to be public?

Do these need to be public?

36 mins ago

Sean Nash

What about TD record CRC errors?

What about TD record CRC errors?

39 mins ago

Sean Nash

Are we only doing DD messages? And since these could be coming from UI now, these shouldn't be considered "Dialin" messages anymore and should be moved up to section before 0x8000 IDs where normal ...

Are we only doing DD messages?
And since these could be coming from UI now, these shouldn't be considered "Dialin" messages anymore and should be moved up to section before 0x8000 IDs where normal system messages are defined.

47 mins ago

Sean Nash

Remove blank line.

Remove blank line.

48 mins ago

Sean Nash

Move this transition function call to the transitionToPostTreatmentMode function below.

Move this transition function call to the transitionToPostTreatmentMode function below.

50 mins ago

Sean Nash

Invalid payload length shouldn't be a reject reason, it should cause a s/w fault (because it suggests the UI and TD software are not compatible versions).

Invalid payload length shouldn't be a reject reason, it should cause a s/w fault (because it suggests the UI and TD software are not compatible versions).

53 mins ago

Sean Nash

Still not what I had in mind. Let's discuss this.

Still not what I had in mind. Let's discuss this.

Thursday 23 Apr
11:11 am

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

11:10 am

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

11:10 am

Vendor - TEL - Varshini Nagabooshanam

updated

updated

11:10 am

Vendor - TEL - Varshini Nagabooshanam

added

added

11:10 am

Vendor - TEL - Varshini Nagabooshanam

Added

Added

11:09 am

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

11:09 am

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

Wednesday 22 Apr
2:05 pm

Sean Nash

Not necessary

Not necessary

2:05 pm

Sean Nash

This command should be given once on way out of prior state instead of every 50ms while in this state. We also need to zero the timer counter at same time so the timeout check below works.

This command should be given once on way out of prior state instead of every 50ms while in this state.
We also need to zero the timer counter at same time so the timeout check below works.

2:03 pm

Sean Nash

Response needs to be sent immediately (in message handler function). This is too late.

Response needs to be sent immediately (in message handler function). This is too late.

2:02 pm

Sean Nash

We shouldn't need to do this door check if you've required door be closed for the state. Call the doorClosedRequired( TRUE ) function on transition into a state that requires the door to be closed ...

We shouldn't need to do this door check if you've required door be closed for the state.
Call the doorClosedRequired( TRUE ) function on transition into a state that requires the door to be closed and call with ( FALSE ) on transition into a state that doesn't require door to be closed. When required = TRUE, the switch monitor will trigger the door open alarm for you.

1:57 pm

Sean Nash

Same organization comment that I had for post-tx mode. Let's keep functions related to each pre-tx state together separated by some kind of comment banner.

Same organization comment that I had for post-tx mode. Let's keep functions related to each pre-tx state together separated by some kind of comment banner.

1:56 pm

Sean Nash

Remove extra blank line.

Remove extra blank line.

1:56 pm

Sean Nash

Looks like there's plans for 4 post-tx states and we only have your auto eject state implemented here below. Since all of these 4 states will be implemented in this file, maybe we should organize t...

Looks like there's plans for 4 post-tx states and we only have your auto eject state implemented here below.
Since all of these 4 states will be implemented in this file, maybe we should organize them - keep functions related to each state together separated by some kind of comment banner.

1:53 pm

Sean Nash

Not necessary if you give retract command on way out of prior state.

Not necessary if you give retract command on way out of prior state.

1:52 pm

Sean Nash

I don't think this if statement is necessary. We wouldn't be in this state if the ejector didn't extend already. I think we just want to see if BP is home again (completed 1 revolution) so we know ...

I don't think this if statement is necessary. We wouldn't be in this state if the ejector didn't extend already.
I think we just want to see if BP is home again (completed 1 revolution) so we know when to stop and go to next state.

1:48 pm

Sean Nash

Give ejector retract command and zero timer counter here too.

Give ejector retract command and zero timer counter here too.

1:47 pm

Sean Nash

Not needed if you do this on way out of prior state.

Not needed if you do this on way out of prior state.