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

Extension for nuXmv #1

Open
haspltom opened this issue Aug 7, 2020 · 5 comments
Open

Extension for nuXmv #1

haspltom opened this issue Aug 7, 2020 · 5 comments

Comments

@haspltom
Copy link

haspltom commented Aug 7, 2020

Hello,

nice tool and features!

Is it planned to extend the tool to be able to handle xmv-output as well?
Or if not, can you may make suggestions how to tackle this the best way?

Best regards
Thomas

@igor-buzhinsky
Copy link
Owner

igor-buzhinsky commented Aug 8, 2020

Hi Thomas,

Thanks!

In simple cases, nuXmv output will be accepted due to having the same format as the one of NuSMV. If you need some nuXmv-specific features, the answer depends on these exact features. Adding support of rationals would be relatively easy. Adding support of timed traces would be more troublesome, but I believe that it is possible.

What exactly would you like to be supported? It it is a simple thing, maybe I can add it. If it is less simple, I can advise which pieces of code need to be modfied to implement it.

This tool is no longer actively developed. On the other hand, there is another tool: https://github.com/ShakeAnApple/cxbacktracker, which is. In short, it explains counterexamples not only on the formula, but also on the diagram of modules. nuXmv support is one of the options for future development of this tool.

@haspltom
Copy link
Author

Hello.

Thanks for your reply.

We tried to use the tool with a nuXmv output and the error message we got, was that real values are not supported.
So, this would be the first feature we favor.
Appart from that, it is hard to say, what else features would be needed as we do not get over this first error message.
We definitely do not use timed nuXmv currently.

Thanks anyways for your suggestion for the other tool, we are currently trying this as well.

Best regards
Thomas

@igor-buzhinsky
Copy link
Owner

Hi,

I added support of reals. Files like this should now be supported: https://github.com/igor-buzhinsky/nusmv_counterexample_visualizer/blob/master/example-input-nuxmv-reals.txt
However, I haven't thoroughly tested this feature, I am going to do this on Wednesday. If there are things that I am missing, please let me know.

@igor-buzhinsky
Copy link
Owner

Hi,

I haven't found any more bugs. Reals should work.

@haspltom
Copy link
Author

Very cool, thanks!
Simple nuXmv output with reals works fine now.

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

No branches or pull requests

2 participants