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

Major Cleanup of the PChecker #784

Closed
wants to merge 21 commits into from
Closed

Conversation

ankushdesai
Copy link
Member

@ankushdesai ankushdesai commented Oct 7, 2024

This is a major clean up/ reorganization of PChecker code, a summary of what gets cleaned up:

  • Actor and PMachine classes are merged into StateMachine class
  • ActorRuntime and CoyoteRuntime class all merged into ControlledRuntime
  • PEvent merged with Event class
  • PMonitor class merged into Monitor class
  • Removed unnecessary classes and functions relating to changes above
  • Reorganized PChecker/CheckerCore folder

ankushdesai and others added 16 commits May 3, 2024 13:42
fixing actor comment

fixing

Co-authored-by: Christine Zhou <[email protected]>
Co-authored-by: Ankush Desai <[email protected]>
* [Cleanup] Moving ActorRuntime into ControlledRuntime

* Changing all references of actor to state machine

---------

Co-authored-by: Christine Zhou <[email protected]>
…g PLogFormatter to PCheckerLogTextFormatter (#763)

Co-authored-by: Christine Zhou <[email protected]>
* [Cleanup] Remove GodMachine, reformat PChecker folder, add assert false after raise test case

* [Cleanup] Remove IStateMachineRuntime, ICyoteRuntime, CyoteRuntime

* [Cleanup] Fixing monitor Raise statement + unit tests

---------

Co-authored-by: Christine Zhou <[email protected]>
Organized exceptions files; removed dgml file generation; remove Guid; changed Prt naming files and values to P; Remove Coyote references from comments

Co-authored-by: Christine Zhou <[email protected]>
@ankushdesai ankushdesai requested review from aishu-j and removed request for ChristineZh0u and aishu-j October 7, 2024 20:57
@ankushdesai ankushdesai closed this Oct 9, 2024
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