Ability not to filter out sites on intersection

jeromekelleher / sc2ts

Infer a succinct tree sequence from SARS-COV-2 variation data

MIT License

4 stars 3 forks source link

Ability not to filter out sites on intersection #81

Closed hyanwong closed 1 year ago

hyanwong commented 1 year ago

As the nextstrain JSON doesn't have any sites, so the intersection with that tree results in no sites at all

jeromekelleher commented 1 year ago

LGTM, just needs rebase to drop commit added in #75

hyanwong commented 1 year ago

Done, ready to merge.

hyanwong commented 1 year ago

Done, ready to merge.

Actually, I just added tiny extra functionality to pass kwargs to the simplify command (e.g. to allow unary nodes to be kept, etc, so please check again before merging.