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

Raw verification options issues - fix 2046945 #128

Merged
merged 51 commits into from
Jan 30, 2024

Conversation

mtygesen
Copy link
Contributor

@mtygesen mtygesen commented Dec 28, 2023

Fixes:
https://bugs.launchpad.net/tapaal/+bug/2046945
https://bugs.launchpad.net/tapaal/+bug/2046960

Also fixes:

  • Width of raw verification options textfield when typing in a long string and then saving and reopening
  • Raw verification options showing when trace option was updated
  • Bug where verification options would enable when changing queryType

@srba srba self-requested a review December 28, 2023 15:17
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 the problem but there is another one that would be nice to be fixed in this PR as well. If you open e.g. the ERK example and the query and then click on "deadlock" in the query, the buttons for editing the query are not enabled (only the "not" button is enabled). Probably a refresh of the buttons should be added somewhere.

…ng text would expand the raw verification options field width upon opening and closing the query dialog
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.

When you open the query and click on deadlock, the query editing buttons are still not enabled (only and/or/not). Compare to the correct behaviour in the released version 3.9.5 (can be downloaded from www.tapaal.net).

@srba srba self-requested a review January 9, 2024 17:20
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.

There is a small issue: open the colored referendum example, open the first query and select "Some trace" and "Save and verify". All goes fine. Now open the query again, select "Use" and "Save and verify" again and we get an error: "the PNML file cannot be parsed due to syntax errors." This should not happen - maybe the file has a different name than what the GUI expects?

@srba
Copy link
Member

srba commented Jan 24, 2024

For hyperLTL queries in the query dialog, the options "Use trace abstraction refinement" and "Use siphon-trap analysis" should be both disabled (greyed out).

@srba
Copy link
Member

srba commented Jan 24, 2024

There is a slight problem with a focus for making LTL and hyperLTL queries. To reproduce, open e.g. ERK model,
make a new query select LTL (or hyperLTL), then click on the <> in the query text field, then click on "Use" and press "A" or "E" (path quantifiers). Now the logical operators are enabled but not the predicates - they get enabled first after you click again on <> in the query.
This is not a problem if you do not press "Use" and also not a problem for CTL queries.

@srba
Copy link
Member

srba commented Jan 24, 2024

Also the boundendess check is broken. Open e.g. the intro example, open the query and run "check boundedness". The engine fails because it got the argument "null" from the GUI.

@srba
Copy link
Member

srba commented Jan 25, 2024

There is an issue in game-harddisk. Open the model, open the query, click on "Use", then select the whole query, click on "AF" and now "Add predicate to the query" becomes enabled (but it should not, because if you press it, it will remove the quntifier AF and verification of the query fails.
This issue can be only reproduced if "Use" is pressed. Without it, it works just fine.

@srba
Copy link
Member

srba commented Jan 25, 2024

One more issue to be fixed: In the intro example, when you click on the query (select all), it allows to change the quantification (to EF, AF, EG and AG) which is correct behaviour but it should not allow to change the logic (all three buttons and/or/not should be disabled). They should be enabled only if you click on the IntroExample.Target = 1 only (without the quantification).

@srba
Copy link
Member

srba commented Jan 26, 2024

For CTL/Reachability untimed nets, it should still be allowed to have and/or/not before the temporal operators. This was only a problem for the timed nets like intro-example.

@srba
Copy link
Member

srba commented Jan 27, 2024

For LTL and hyperLTL queries, it should be possible that after pressing "A" and then "U" and then selecting the whole "U" with both place holders, then and/or/not should be enabled, right now this is not possible - compare to 3.9.5 where it allows to add Boolean operators when the until "U" expression is selected.

@srba
Copy link
Member

srba commented Jan 28, 2024

The query creating for CTL/Reachability is broken now. If you make new query (e.g. on ERK) then the Boolean operators are not enabled - but they should (compare with 3.9.5). Also if you make query like EF false and select all and want to add negation before it, then the EF false query disappears - again compare with 3.9.5.

@srba
Copy link
Member

srba commented Jan 29, 2024

The behaviour for LTL and hyperLTL is not correct - it allows to add and/or/not at the highest level, which the engine does not like - compare on e.g. ERK with 3.9.5 which does not allow that.

@srba srba self-requested a review January 30, 2024 16:19
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.

testing all and seems to work fine now, merging to main

@srba srba merged commit c03ada8 into TAPAAL:main Jan 30, 2024
1 check passed
@mtygesen mtygesen deleted the raw-options-issues-2046945 branch January 31, 2024 15:48
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