jeromekelleher / sc2ts

Infer a succinct tree sequence from SARS-COV-2 variation data
MIT License
4 stars 3 forks source link

Always add number of descendant samples #140

Closed hyanwong closed 1 year ago

hyanwong commented 1 year ago

As discussed by Ana. Doesn't look great at the moment with large labels, so needs some tweaking:

Screenshot 2023-03-24 at 09 09 01
hyanwong commented 1 year ago

We can actually remove the "samples" suffix by using the label hack I merged earlier:

sc2ts.sample_subgraph(712029, ts, treeinfo, 
    node_label_replace={
        "Unknown":"",
        "Unknown ":"",
        " samples": ""  ## Remove the word "samples"
    },
   ...
)
plt.show()
jeromekelleher commented 1 year ago

LGTM - @a-ignatieva what do you think?

a-ignatieva commented 1 year ago

Looks good! Thanks @hyanwong