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

Fix accountability examples #607

Merged

Conversation

kevinmorio
Copy link
Contributor

The PR adapts the accountability examples to the new SAPiC semantic.
Moreover, it adds some accountability examples to the regression tests.

@rkunnema
Copy link
Member

Hi!

@kevinmorio and I did some analysis.

#595 introduced some code changes that make the integration tests fail:

 /fast-tests/related_work/YubiSecure_KS_STM12/Yubikey_and_YubiHSM_multiset_analyzed.spthy
The number of rules are not equal!
/fast-tests/related_work/YubiSecure_KS_STM12/Yubikey_and_YubiHSM_multiset_analyzed.spthy
The warnings do not match!

Annoyingly, we cannot reproduce this failure, neither on MacBook Pro 2019, Arch Linux, Debian 10 and Ubuntu 22.04, all with the same Maude that the integration test uses. Hence we cannot upload a file that stops this failure.

This will block other PRs as well (sorry!). The onus on fixing this is on us. We think the best way to deal with this is to extend the github workflow with a way to download the artefacts, so we can debug this issue.

@rkunnema rkunnema merged commit e6a90bf into tamarin-prover:develop Dec 5, 2023
1 check passed
Lelio-Brun pushed a commit to zt-iot/tamarin-prover that referenced this pull request Apr 18, 2024
* Adapted some accountability examples to new SAPIC syntax.

* Adapt another accountability example to updated SAPIC semantic

* Add oracle heuristic to accountability examples

* Adapt accountability examples from Master's thesis to new SAPiC semantic

* Update README

* Add README

* Change to monospace

* Add accountability case studies to regression tests

* Revert failing case study to upstream version

* Fix unintentional restore of failing case study

---------

Co-authored-by: Robert Künnemann <robert.kuennemann@cispa.saarland>
Co-authored-by: rkunnema <robert.kuennemann@cispa.de>
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.

None yet

2 participants