jcrozum / biobalm

The biologist's Boolean attractor landscape mapper, building Waddington landscapes from Boolean networks.
https://jcrozum.github.io/biobalm/
MIT License
2 stars 0 forks source link

Percolation optimizations #86

Closed jcrozum closed 11 months ago

github-actions[bot] commented 11 months ago

Coverage

Coverage Report
FileStmtsMissCoverMissing
nfvsmotifs
   SuccessionDiagram.py156497%6–7, 208, 468
   control.py1201389%47, 56, 60, 66, 80, 89–105, 319, 334
   interaction_graph_utils.py142894%6–9, 57, 70, 95–96
   motif_avoidant.py163597%23–24, 130, 184, 309
   petri_net_translation.py84693%23–24, 52, 63–64, 94
   pyeda_utils.py953464%12, 56–66, 90, 95, 98–112, 140–144
   space_utils.py145696%15–16, 189, 218, 233, 290
   state_utils.py681282%15, 55–66, 98, 105, 114
   terminal_restriction_space.py44491%6–7, 80, 97
   trappist_core.py1862089%10–11, 39, 41, 81, 127, 192, 194, 196, 231–233, 259, 317, 319, 349, 389, 391, 422, 451
nfvsmotifs/FVSpython3
   FVS.py481079%90–91, 97, 133, 183–189
   FVS_localsearch_10_python.py90199%179
nfvsmotifs/_sd_algorithms
   compute_attractor_seeds.py29197%6
   expand_attractor_seeds.py51492%6, 95–100
   expand_bfs.py28196%6
   expand_dfs.py30197%6
   expand_minimal_spaces.py37197%6
   expand_source_SCCs.py184796%84, 94, 137, 160–161, 166, 281
   expand_to_target.py30390%6, 37, 42
TOTAL182714192% 

Tests Skipped Failures Errors Time
366 0 :zzz: 0 :x: 0 :fire: 4m 11s :stopwatch:
jcrozum commented 11 months ago

@kyuhyongpark @daemontus Here is the PR I promised this morning. Feel free to look over it, but be warned that the GitHub website isn't handling all the moved model files very gracefully :)

@giang-trinh for context, this PR has the SCC expansion optimizations @kyuhyongpark was working on (#79), and also a fix for #84. We agreed to merge these into main as soon as possible because the branch structure was getting a bit out of hand.

Edit: forgot to add that there are some analysis scripts in here. We might want to reorganize these later, but for now I think we should just merge them in.

daemontus commented 11 months ago

I also looked at the relevant code and I§m fine with merging this :)