tskit-dev / tsbrowse

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

Mutations popup #188

Closed benjeffery closed 1 month ago

benjeffery commented 1 month ago

Puts the mutation information in a pop-up. Also makes mutation points visible with a hack - root cause is zero inheritors.

https://github.com/user-attachments/assets/b254d18a-bc9f-4995-a822-5f6763a4d4de

github-actions[bot] commented 1 month ago

Automated Screenshots

These screenshots are automatically updated as the PR changes.

Click to view screenshots ### out_edges.png ![out_edges.png](https://raw.githubusercontent.com/tskit-dev/tsbrowse/pr-screenshots/pr_188/out_edges.png) ### out_mutations.png ![out_mutations.png](https://raw.githubusercontent.com/tskit-dev/tsbrowse/pr-screenshots/pr_188/out_mutations.png) ### out_nodes.png ![out_nodes.png](https://raw.githubusercontent.com/tskit-dev/tsbrowse/pr-screenshots/pr_188/out_nodes.png)
coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 11293839171

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
tsbrowse/pages/mutations.py 8 10 80.0%
<!-- Total: 10 12 83.33% -->
Files with Coverage Reduction New Missed Lines %
tsbrowse/plot_helpers.py 2 53.49%
<!-- Total: 2 -->
Totals Coverage Status
Change from base Build 11233768292: -0.4%
Covered Lines: 679
Relevant Lines: 825

💛 - Coveralls
jeromekelleher commented 1 month ago

Super! That looks like it works a lot better than second panel?

savitakartik commented 1 month ago

Wow, this is so neat! 😍

benjeffery commented 1 month ago

The colour scheme for mutations was white for mutations with low numbers of inheritors - I've change it to one that goes green to blue.