fwcommon

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merge branch 'DEN-14646-fpga-alarm-properties-and-timing-3' into staging

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

Corrected

Corrected

Corrected

Corrected

Corrected

Corrected

Does TODO need to be resolved?

Does TODO need to be resolved?

Extra space before the last paren.

Extra space before the last paren.

Missing space.

Missing space.

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

I think VPi is closed during standby mode, so not sure why we want to check inlet conductivity in this mode. I imagine new system would get installed by service - would need some kind of priming do...

I think VPi is closed during standby mode, so not sure why we want to check inlet conductivity in this mode.
I imagine new system would get installed by service - would need some kind of priming done in service mode, so I'm not worried about that.

will these alarms trigger with open air? Ex: if a user has a device shipped to them, will they be greeted with conductivity alarms on first power up before any water has ran through the system? If...

will these alarms trigger with open air?

Ex: if a user has a device shipped to them, will they be greeted with conductivity alarms on first power up before any water has ran through the system? If so, how will they be able to fix it?

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Merge branch 'DEN-14999-alarm-table-updates' into develop

DEN-14999, DEN-15031 reused the start DG structure that already exists

DEN-14999, DEN-15031 updated the DG NV records with the acid and bicarb values

DEN-14646 Code cleanup

DEN-14999, DEN-15031 updated the NV data management to add the fill conductivity values.

I don't see it, but resolving anyway. RESOLVED in CODE WALKTHROUGH.

I don't see it, but resolving anyway.
RESOLVED in CODE WALKTHROUGH.

DEN-14999 added another condition in NV data management to make sure all the jobs in the queue are processed and the state machine is in idle state so even the last queue has been fully read

Spaces inside (). And explicit condition (i.e. ( TRUE == isConvNotValid ).

Spaces inside (). And explicit condition (i.e. ( TRUE == isConvNotValid ).

DEN-14646 Simplify checkFPGAPersistentAlarms

Corrected

Corrected

Merge branch 'DEN-14999-alarm-table-updates' into develop

DEN-14999 added a data structure for the DG start command

Merge branch 'staging' into DEN-14646-fpga-alarm-properties-and-timing-3