Overall sequence timeout duration is missing in library

equinor / iec63131

Other

9 stars 8 forks source link

Overall sequence timeout duration is missing in library #53

Closed AlexTxen closed 1 year ago

AlexTxen commented 2 years ago

Overall sequence timeout is to be added to library. Proposal is to add 'TMax' (with 'Parameter' type) attribute to 'StandardSequenceElementClass' -> 'Start' class. It is assumed, that alarm priority should be on YF terminal of corresponding KB block (already in library).

cdenisey commented 1 year ago

Discussions have been taken in the workshop held on 21.06.2023, refer to item 2. https://github.com/equinor/iec63131/blob/2c8fa87426e072459c5afc61afd97b1ce53cd28c/MOM%20AML%20Library%200.0.11%20workshop.docx

It has been agreed to add the overall sequence timeout attributes (‘TMax’ attribute and ‘SequenceFlowTimeout’ interface: Annouciation attribute and linking to End or specific step)

AlexTxen commented 1 year ago

Implemented in 2ccc450d5b6fb97802fd75b3b88842dd788ee27a Issue can be closed after registering separate issue on Continue/Hold mechanism implementation,

Added 'TMax' attribute to sequence 'Start' class:

Added 'SequenceFlowTimeout' interface to sequence 'Start' class. Interface connection to step/end represents sequence flow on timeout. 'Annunciation' attribute represents alarming options.