tskit-dev / tsbrowse

Utilities for evaluating inferred tree sequences
MIT License
3 stars 8 forks source link

Blank mutations plot caused by all mutations being "unknown" #185

Open benjeffery opened 6 days ago

benjeffery commented 6 days ago

We should alert the user somehow when mutation times are unknown. This has caused some of the confusion over at #179, where old .trees files from the unified genealogy have no mutation time.

benjeffery commented 6 days ago

I note at https://github.com/tskit-dev/tsbrowse/blame/1f3bfbd77b9a66725d9399a64fa64059e5f002d7/tsbrowse/model.py#L418 @jeromekelleher suggests using the compute mutation times routine from tskit.

jeromekelleher commented 6 days ago

Emitting a warning and computing times seems the right approach