GUI: Support eq length and range weighted tree plots #912
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This also caches.
This will simplify the next step of adding range weighted eq branch length trees.
This means other packages don't need to inherit all of B::C just to get caching support.
Makes little difference for small trees but will matter for large ones.
We were previously switching back to length mode, leaving the menu out of synch.
These were being triggered when the project level tree was changed, but the spatial tab gets its tree from the combobox.
Avoiding some looping and method calls will help with larger trees.
Might make a small speed difference.
We are assigning anyway so "no autoviv" has no benefit as all intermediate keys are created if needed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updates #911