Generalizing 'collapse_mutationless_edges' and small fitchcount bugfix

[richardyz98]

This pull request includes the generalization of the collapse_mutationless_edges function. This allows us to use a function that does not factor in missing data when determining whether to collapse two nodes. If we do not collapse on missing data, then the baseline score increases for trees as missing data increases.

Additionally, it fixes a small bug in fitchcount which can error if no root is provided, such as on an initial call of the function.

[codecov]

Codecov Report

Merging #181 (8a3d4d9) into master (0bd3126) will decrease coverage by 0.22%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #181      +/-   ##
==========================================
- Coverage   87.40%   87.18%   -0.23%     
==========================================
  Files          72       72              
  Lines        5631     5633       +2     
==========================================
- Hits         4922     4911      -11     
- Misses        709      722      +13     

Impacted Files	Coverage Δ
cassiopeia/data/CassiopeiaTree.py	91.50% <100.00%> (+0.02%)	⬆️
cassiopeia/tools/small_parsimony.py	98.51% <100.00%> (ø)
cassiopeia/solver/HybridSolver.py	75.00% <0.00%> (-9.49%)	⬇️
cassiopeia/solver/MaxCutGreedySolver.py	93.47% <0.00%> (-4.35%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0bd3126...8a3d4d9. Read the comment docs.