Skip to content

[ENH] canvas: Indicate runtime state on links#1554

Merged
janezd merged 2 commits intobiolab:masterfrom
ales-erjavec:canvas/display-state
Sep 21, 2016
Merged

[ENH] canvas: Indicate runtime state on links#1554
janezd merged 2 commits intobiolab:masterfrom
ales-erjavec:canvas/display-state

Conversation

@ales-erjavec
Copy link
Contributor

Track and display the state of signal propagation on links in the canvas.

@codecov-io
Copy link

codecov-io commented Sep 9, 2016

Current coverage is 88.69% (diff: 100%)

Merging #1554 into master will not change coverage

@@             master      #1554   diff @@
==========================================
  Files            78         78          
  Lines          8108       8108          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           7191       7191          
  Misses          917        917          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 7ef8aaa...996a236

@astaric
Copy link
Member

astaric commented Sep 12, 2016

Have you considered switching the state that is shown on widgets? (Like highlighting the channels that do not pass any data)?

Copy link
Contributor

@BlazZupan BlazZupan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice idea! I propose to mark the connections with no tokens (None) and leave all connections with tokens as they are (grey). Connection with no tokens can be shown as broken, the way we show now the connections that are disabled. For disabled connections use "a cut line" symbol: break connection in the middle and display the cut with two lines, something like ---//--- (except that the line is full and the only break really is between //)

* Disabled links are now displayed with a single disconnected symbol in
  the middle.
* The dash line style is re-used for links with no data on them.
Copy link
Member

@astaric astaric left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the way it looks now, and would really like to see this change merged.

Copy link
Contributor

@ajdapretnar ajdapretnar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it too. Approve for Windows.

@janezd
Copy link
Contributor

janezd commented Sep 21, 2016

It looks great.

@janezd janezd merged commit 9730c7a into biolab:master Sep 21, 2016
@janezd janezd changed the title [RFC] [ENH] canvas: Indicate runtime state on links [ENH] canvas: Indicate runtime state on links Sep 21, 2016
@ales-erjavec ales-erjavec deleted the canvas/display-state branch September 28, 2016 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants