MBP-247: Position Recovery Improvements - #44
Conversation
Adds an additional restore position state to E_RestorePosition.
Add persistent variable in the GVL to store the current date and time at shutdown.
Add homing status of axis before shutdown.
ST_AxisPersistent is extended to include axis status variables to be saved before power failure.
Remove the enums eRestoreWithHome and eRestoreWithoutHome.
Only keep the persistent variables that are required for the position recovery.
szilard-ess
left a comment
szilard-ess
left a comment
There was a problem hiding this comment.
While I was working on a different project, I encountered that the following comment in ST_AxisConfig is not correct. This should be updated for the new process, the end of the comment implies a different type.
eRestorePosition: E_RestorePosition; //Option to restore incremental axis position after power loss(0 = No, 1 = Yes)
DUTs/Axis_Structures/ST_AxisConfig - Line 12
Update the description of the E_RestorePosition in ST_AxisConfig and the DUT.
Uses encoder reference system to decide restore type. Keeps only the check for whether restore is needed.
Remove double semicolon from bAxisHasBeenRestored.
szilard-ess
left a comment
szilard-ess
left a comment
There was a problem hiding this comment.
Tested and works, see PR17 of tc_generic_structure for more info
|
HI @aslanliafichev , I remember a lot of back and forth about the requirements of this ticket. That we discover that some functionality described in the requirements in the ticket was not necessary or it did not make sense functional wise. Could you please describe here: https://confluence.ess.eu/spaces/MCAG/pages/952113567/Position+Recovery+PROG what is the current functionality of the recovery system, with which kind of reference systems it works and what recovery options we have. |
4 participants
No description provided.