leahi-fwcommon

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Fixed.

Fixed.

Moved.

Moved.

Move buttons down just above "Comm.h". And why is buttons being included here?

Move buttons down just above "Comm.h". And why is buttons being included here?

This reason code is not used anywhere, and the existing REQUEST_REJECT_REASON_NOT_ALLOWED_IN_CURRENT_MODE can be used if a rejection response is needed. I removed the new reason code.

This reason code is not used anywhere, and the existing REQUEST_REJECT_REASON_NOT_ALLOWED_IN_CURRENT_MODE can be used if a rejection response is needed. I removed the new reason code.

I don't see it.

I don't see it.

I don't see it.

I don't see it.

I would think reason code 1 would work for this - do we really need to add this one?

I would think reason code 1 would work for this - do we really need to add this one?

Fix this.

Fix this.

Updated as suggested by removing the unnecessary else block and reject command. Retested the power-off confirmation and shutdown flow successfully.

Updated as suggested by removing the unnecessary else block and reject command. Retested the power-off confirmation and shutdown flow successfully.

Updated to remove the return from the middle of the function. The function now sets newID and returns only once at the end. Retested successfully.

Updated to remove the return from the middle of the function. The function now sets newID and returns only once at the end. Retested successfully.

renamed. thanks!

renamed. thanks!

renamed

renamed

updated to not have it update always

updated to not have it update always

updated with latest staging implementation.

updated with latest staging implementation.

Added but crucible is not showing this change. Please check bitbucket.

Added but crucible is not showing this change. Please check bitbucket.

These variables are correctly initialized to the first enum in the list.

These variables are correctly initialized to the first enum in the list.

Not needed because the retry mechanism is going to be deleted in the next ticket.

Not needed because the retry mechanism is going to be deleted in the next ticket.

I would rather keep it. Anyways, we need an else block after else if, as per MISRA guidelines.

I would rather keep it. Anyways, we need an else block after else if, as per MISRA guidelines.

I would rather keep it. Anyways, we need an else block after else if, as per MISRA guidelines.

I would rather keep it. Anyways, we need an else block after else if, as per MISRA guidelines.

Not needed because the retry mechanism is going to be deleted in the next ticket.

Not needed because the retry mechanism is going to be deleted in the next ticket.

What is happening here? Not clear if we need cartridge param. Let's remove it for now.

What is happening here? Not clear if we need cartridge param. Let's remove it for now.

Remove extra blank line.

Remove extra blank line.

Remove extra blank line.

Remove extra blank line.

Comment these out again.

Comment these out again.

Move this to right before syringe pump controller below. Add comment "Execute pinch valve command state machine."

Move this to right before syringe pump controller below. Add comment "Execute pinch valve command state machine."

Move this to right after valves controller exec below.

Move this to right after valves controller exec below.

Restore this.

Restore this.

Restore this.

Restore this.

uncomment this line.

uncomment this line.

Remove test code.

Remove test code.