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

Control of succession #73

Closed
wants to merge 14 commits into from
Closed

Control of succession #73

wants to merge 14 commits into from

Conversation

jcrozum
Copy link
Owner

@jcrozum jcrozum commented Jun 20, 2023

This PR implements succession diagram control. Most basic features are covered, though we may need to do some work to make sure this is all compatible with #71.

See #69 for additional details.

@jcrozum jcrozum requested a review from daemontus June 20, 2023 18:43
@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
nfvsmotifs
   SuccessionDiagram.py1411987%6, 59, 67–70, 104, 114–115, 176, 222, 232, 236, 246, 250, 334–337, 389
   control.py1221389%47, 56, 60, 66, 80, 89–105, 327, 342
   interaction_graph_utils.py142894%6–9, 57, 70, 95–96
   motif_avoidant.py116497%25, 58, 72, 110
   petri_net_translation.py84693%23–24, 52, 63–64, 94
   pyeda_utils.py953464%12, 56–66, 90, 95, 98–112, 140–144
   space_utils.py1101487%15–16, 36–43, 52, 198, 213, 270
   state_utils.py681282%15, 55–66, 98, 105, 114
   terminal_restriction_space.py44393%6–7, 80
   trappist_core.py1862089%10–11, 39, 41, 81, 122, 182, 184, 186, 221–223, 249, 306, 308, 338, 378, 380, 411, 440
nfvsmotifs/FVSpython3
   FVS.py481079%90–91, 97, 133, 183–189
   FVS_localsearch_10_python.py90199%179
TOTAL134314489% 

Tests Skipped Failures Errors Time
227 0 💤 0 ❌ 0 🔥 2m 17s ⏱️

@jcrozum
Copy link
Owner Author

jcrozum commented Jul 25, 2023

This PR is not compatible with the recently merged #71. I will make a new PR that is, but I'm leaving this up in the meantime to help me keep track of things.

jcrozum added a commit that referenced this pull request Jul 25, 2023
@jcrozum jcrozum closed this in #74 Aug 1, 2023
jcrozum added a commit that referenced this pull request Aug 1, 2023
@jcrozum jcrozum deleted the control-of-succession branch August 2, 2023 18:15
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

1 participant