Skip to content
This repository was archived by the owner on Apr 25, 2024. It is now read-only.

Move kcfg commit logic to KCFG class#909

Merged
rv-jenkins merged 5 commits intomasterfrom
noah/kcfg-extend
Feb 22, 2024
Merged

Move kcfg commit logic to KCFG class#909
rv-jenkins merged 5 commits intomasterfrom
noah/kcfg-extend

Conversation

@nwatson22
Copy link
Copy Markdown
Contributor

  • KCFGExplore.extend_kcfg() replaced by KCFG.extend()
  • Logging of ExtendResults moved earlier to when they are created (before, they were logged when applied, but this is no longer possible with application logic (extend()) living in KCFG.
  • KCFGExplore.extend() removed and its logic moved up into advance_pending_node()

@nwatson22 nwatson22 self-assigned this Feb 22, 2024
@nwatson22 nwatson22 requested review from ehildenb and tothtamas28 and removed request for ehildenb February 22, 2024 19:52
@nwatson22 nwatson22 marked this pull request as ready for review February 22, 2024 20:38
@nwatson22 nwatson22 added the enhancement New feature or request label Feb 22, 2024
Comment thread src/pyk/kcfg/kcfg.py Outdated
@rv-jenkins rv-jenkins merged commit b4c8c96 into master Feb 22, 2024
@rv-jenkins rv-jenkins deleted the noah/kcfg-extend branch February 22, 2024 21:26
virgil-serbanuta pushed a commit that referenced this pull request Feb 23, 2024
- `KCFGExplore.extend_kcfg()` replaced by `KCFG.extend()`
- Logging of `ExtendResult`s moved earlier to when they are created
(before, they were logged when applied, but this is no longer possible
with application logic (`extend()`) living in `KCFG`.
- `KCFGExplore.extend()` removed and its logic moved up into
`advance_pending_node()`

---------

Co-authored-by: devops <devops@runtimeverification.com>
Baltoli pushed a commit to runtimeverification/k that referenced this pull request Apr 9, 2024
- `KCFGExplore.extend_kcfg()` replaced by `KCFG.extend()`
- Logging of `ExtendResult`s moved earlier to when they are created
(before, they were logged when applied, but this is no longer possible
with application logic (`extend()`) living in `KCFG`.
- `KCFGExplore.extend()` removed and its logic moved up into
`advance_pending_node()`

---------

Co-authored-by: devops <devops@runtimeverification.com>
Baltoli pushed a commit to runtimeverification/k that referenced this pull request Apr 9, 2024
- `KCFGExplore.extend_kcfg()` replaced by `KCFG.extend()`
- Logging of `ExtendResult`s moved earlier to when they are created
(before, they were logged when applied, but this is no longer possible
with application logic (`extend()`) living in `KCFG`.
- `KCFGExplore.extend()` removed and its logic moved up into
`advance_pending_node()`

---------

Co-authored-by: devops <devops@runtimeverification.com>
Baltoli pushed a commit to runtimeverification/k that referenced this pull request Apr 10, 2024
- `KCFGExplore.extend_kcfg()` replaced by `KCFG.extend()`
- Logging of `ExtendResult`s moved earlier to when they are created
(before, they were logged when applied, but this is no longer possible
with application logic (`extend()`) living in `KCFG`.
- `KCFGExplore.extend()` removed and its logic moved up into
`advance_pending_node()`

---------

Co-authored-by: devops <devops@runtimeverification.com>
Baltoli pushed a commit to runtimeverification/k that referenced this pull request Apr 10, 2024
- `KCFGExplore.extend_kcfg()` replaced by `KCFG.extend()`
- Logging of `ExtendResult`s moved earlier to when they are created
(before, they were logged when applied, but this is no longer possible
with application logic (`extend()`) living in `KCFG`.
- `KCFGExplore.extend()` removed and its logic moved up into
`advance_pending_node()`

---------

Co-authored-by: devops <devops@runtimeverification.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automerge enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants