-
Notifications
You must be signed in to change notification settings - Fork 17
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
Tabakov-Vardi generator #468
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems good. It is a good idea to have some generators in the library itself.
Co-authored-by: David Chocholatý <chocholaty.david@protonmail.com>
What is the state of this PR? Are you working in anything else, or is it considered final and ready for merging? |
It's the final version, and it is ready to be merged. |
@jurajsic What do you think? Anything more to improve? Are you OK with the current solution to the casting between |
* Tabakov-Vardi generator * big test * renaming * Update include/mata/nfa/builder.hh Co-authored-by: David Chocholatý <chocholaty.david@protonmail.com> * renaming + new unique state/transition generator * fin_state_density range changed to [0;1] * paper * float -> double + off-by-one potential error fix * comment --------- Co-authored-by: David Chocholatý <chocholaty.david@protonmail.com>
This PR adds Tabakov-Vardi random automata generator and corresponding tests.