Skip to content

Conversation

@mtygesen
Copy link
Contributor

@mtygesen mtygesen commented Oct 8, 2025

@srba srba self-requested a review October 8, 2025 19:30
Copy link
Member

@srba srba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The fix is wrong, if you open the net from this bug report

https://bugs.launchpad.net/tapaal/+bug/2127171

and ask the LTL query

E (G ((F TAPN1.P6 = 1) and (F TAPN1.P5 = 1)))

using Tarjen, it gives a finite trace which is wrong - compare with the trace returned without Tarjan that is correct.

@srba srba self-requested a review October 9, 2025 20:57
Copy link
Member

@srba srba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The attached query on the model returns a trace that is not executable in the net.

test1.tapn.zip

@srba srba self-requested a review October 16, 2025 12:59
Copy link
Member

@srba srba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes all the issues but there is a new bug introduce - see the attached net and run the second query - this gives a trace that is not executable. This works correctly in the current main, so it was introduced in this PR.
test2.tapn.zip

@srba srba self-requested a review October 22, 2025 15:16
Copy link
Member

@srba srba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The trace changes in the PR also cause wrong verifications answers for LTL with Tarjan, see this bug report
https://bugs.launchpad.net/tapaal/+bug/2131082

@srba srba self-requested a review November 13, 2025 08:50
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