Skip to content

SubsplitDAG: Remove Nodes#435

Draft
davidrich27 wants to merge 10 commits intomainfrom
428-subsplitdag-remove-nodes
Draft

SubsplitDAG: Remove Nodes#435
davidrich27 wants to merge 10 commits intomainfrom
428-subsplitdag-remove-nodes

Conversation

@davidrich27
Copy link
Collaborator

@davidrich27 davidrich27 commented Apr 22, 2022

Description

Work in Progress

  • ArgsortVector / Reindexer
    • SortedInsert
    • SortedDelete
  • SubsplitDAG
    • RemoveNodes
    • AddNodes

Closes #428

Tests

Please describe the tests added to verify correct behavior.
Please double check that you can make your tests fail by perturbing them.

Checklist:

  • Code follows our detailed contribution guidelines
  • clang-format has been run
  • TODOs have been eliminated from the code
  • Comments are up to date, document intent, and there are no commented-out code blocks
  • Issue branch has been squashed and rebased on main branch
  • GitHub CI build on PR branch completed successfully

This issues adds optional flags to `SBNInstance` functions
`PhyloGradients` and `Likelihoods` in the class `PhyloFlags`.
* Option flags are stored as members of `PhyloFlagOptions`.
* Keys for the return type `GradientMap` are stored as members of `PhyloFlagKeys`.
* Options and Keys have been exposed to the pybind through `bito.phylo_flags` and `bito.phylo_keys`.
* There are three flags "types": inclusive (run only if listed), exclusive (DON'T run only if listed) and set (change input value).
Flags can be considered inclusive if not prepended by exclude_ or set_.
* There is a special run_all flag which when set considers all inclusiveflags true.
* `PhyloFlags` can be passed as a list of flags (if there are no set flags), or as a list of (flag, double) tuples.

Closes #363
@davidrich27 davidrich27 linked an issue Apr 22, 2022 that may be closed by this pull request
@davidrich27 davidrich27 self-assigned this Apr 22, 2022
@davidrich27 davidrich27 force-pushed the 428-subsplitdag-remove-nodes branch from 8acc1b9 to 4721de3 Compare May 24, 2022 17:17
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.

SubsplitDAG: Remove node pair.

1 participant