Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change comparison for new IO data according to specification #511

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tinbie
Copy link

@tinbie tinbie commented Jun 6, 2024

A received packet is considered to be new data whenever the CIP Sequence Count is
different (either greater or lesser) than the previous packet’s CIP Sequence Count, Vol2_1.33 3-4.1

…ta, if CIP sequence count is different than previous one, Vol2_1.33 3-4.1
@MartinMelikMerkumians
Copy link
Member

Hi,

I am not aware on any recent changes on the logic for class 0/1 messages. Have you tested this with the recent CT Tool or has there been issues somewhere else?

For quite some time a new package is considered to reset the timeout watchdog, but ordering of messages is still relevant for the update of data

Cheers,
Martin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants