Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

spatial tab - colour tree branches using calculated index when hovering over cells #626

Closed
2 tasks done
shawnlaffan opened this issue Nov 6, 2016 · 1 comment
Closed
2 tasks done

spatial tab - colour tree branches using calculated index when hovering over cells #626

shawnlaffan opened this issue Nov 6, 2016 · 1 comment
Assignees
@shawnlaffan
Labels
GUI
Milestone
Release_2.0

Comments

@shawnlaffan
Copy link
Owner

shawnlaffan commented Nov 6, 2016 •
edited
Loading

In the spatial tab, it would be useful to colour the dendrogram branches using one of the calculated indices.

This would be an alternative to the current scheme of blue/red/black for branches found in one or both neighbour sets.

to do:

  • Show the combo once an analysis has been run
  • Add a legend to the tree pane
@shawnlaffan shawnlaffan added the GUI label Nov 6, 2016
@shawnlaffan shawnlaffan self-assigned this Nov 6, 2016
@shawnlaffan shawnlaffan changed the title spatial tab - colour tree branches using calculated index spatial tab - colour tree branches using calculated index when hovering over cells Nov 6, 2016
shawnlaffan added a commit that referenced this issue Nov 7, 2016
@shawnlaffan
Initial colouring based on lists
6103417 
Currently only does blue.

Updates #626
shawnlaffan added a commit that referenced this issue Nov 7, 2016
@shawnlaffan
Use black for ancestral branches that are not in the selected list
46d90cf 
This way the full path is highlighted,
but colours are only assigned as
to list members.

Updates #626
shawnlaffan added a commit that referenced this issue Nov 7, 2016
@shawnlaffan
Now we colour the branches using the selected index
ca9d09d 
Currently scales as ln(x+1), with no check for zero.
This needs to be changed.

Updates #626
shawnlaffan added a commit that referenced this issue Nov 7, 2016
@shawnlaffan
Add a separator to the bottom toolbar
a6e2151 
Updates #626
shawnlaffan added a commit that referenced this issue Nov 8, 2016
@shawnlaffan
Matrices do not have element lists.
46aeddc 
Updates #626
@shawnlaffan shawnlaffan added this to the Release_2.0 milestone Feb 2, 2017
shawnlaffan added a commit that referenced this issue Aug 14, 2017
@shawnlaffan
Update the dendrogram list selector on run
c46cf95 
updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
Add initial code for a legend in the tree pane
1d47ffc 
Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
show/hide the dendrogram legend depending on the chosen list
2429d53 
Triggers on update, which matches the current
line colour behaviour

Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
Use the legend to control the colouring
5cc9e13 
Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
Use a natural sort for the dendro colouring list
b0c6a88 
Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
Simplify the branch colouring logic
7652a75 
Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
Add menu for tree options, incl log scaling colours
efdbb6b 
Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
formatting only
337ae47 
Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
Implement methods to change colour scheme
79ff6a5 
Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
add method to get at legend width
5063298 
As opposed to direct use of a constant.
This will allow it to vary.

Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
generate the dendrogram legend only when asked
caed562 
And some cleanup

Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
methods to get and set the legend width
f60c600 
Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
reinstate some code
0139b6b 
but be conditional within it

Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
implement the legend show/hide code from the menu option
4c9d458 
Updates #626
shawnlaffan added a commit that referenced this issue Aug 27, 2017
@shawnlaffan
see if we can get a space for the legend
e811c2b 
Commented out for now as it does not
work with zooming.

Updates #626
shawnlaffan added a commit that referenced this issue Aug 29, 2017
@shawnlaffan
code formatting
d17398b 
Updates #626
shawnlaffan added a commit that referenced this issue Aug 29, 2017
@shawnlaffan
Shift the root node to allow for presence of a legend
f9e3a9b 
Updates #626
shawnlaffan added a commit that referenced this issue Aug 29, 2017
@shawnlaffan
Adjust the tree root position to allow for a legend
605f901 
If there is no legend then there is no adjustment.

We could also adjust it when the legend is hidden,
but then we hit redraw issues (the whole tree is rebuilt
on size changes).

Updates #626
shawnlaffan added a commit that referenced this issue Aug 30, 2017
@shawnlaffan
comment out thinner legend width default
7fc8544 
Should really allow user control

Updates #626
shawnlaffan added a commit that referenced this issue Sep 19, 2017
@shawnlaffan
Make sure we show the tree colour combo when spatial analysis is firs…
a95afb8 
…t run

Updates #626
shawnlaffan added a commit that referenced this issue Sep 19, 2017
@shawnlaffan
And ensure we refresh the combo on re-run
16b2d5d 
Updates #626
@shawnlaffan
Copy link
Owner Author

Mark as fixed

shawnlaffan added a commit that referenced this issue Oct 4, 2017
@shawnlaffan
Delete old widgets on refresh
09b616f 
Otherwise we end up with a chain of duplicates on re-runs

updates #626
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shawnlaffan shawnlaffan

Labels
GUI
Projects
None yet
Milestone
Release_2.0
Development

No branches or pull requests
1 participant
@shawnlaffan