Add BCT metrics projection widget - #103
Conversation
| "outputs": [], | ||
| "source": [ | ||
| "from tvb.datatypes.connectivity import Connectivity\n", | ||
| "from tvbwidgets.ui.bct.bct_metrics_widget import BCTMetricsProjectionWidget\n", |
There was a problem hiding this comment.
Maybe you could expose BCTMetricsProjectionWidget in tvbwidgets.api and import it from there here in the notebook, to stay consistent with the other widgets examples.
| "outputs": [], | ||
| "source": [ | ||
| "from tvb.datatypes.connectivity import Connectivity\n", | ||
| "from tvbwidgets.api import BCTMetricsProjectionWidget\n", |
There was a problem hiding this comment.
The reverse order of imports should avoid the library issue:
from tvbwidgets.api import BCTMetricsProjectionWidget
from tvb.datatypes.connectivity import Connectivity
|
As the default connectivity does not have the main diagonal 0, and this conflicts with some analyzers, could we add a step to make the diagonal zero at a user-action? Maybe a button in the widget, or a cell in the notebook ? |
|
I merged another PR recently in |
Currently, the widget checks whether the connectivity matrix has non-zero diagonal entries (which conflict with all analyzers). If so, it displays a confirmation dialog asking the user whether to set the diagonal to zero, and only proceeds with the modification if the user confirms. Would this be considered the intended user-triggered step, or would you prefer a separate Zero Diagonal button instead? |
df24a7e to
591faf5
Compare
|
|
Dear @deeptive, the widget looks good!
|



Adds a BCT metrics projection widget for interactive execution and visualization of Brain Connectivity ToolBox metrics on TVB Connectivity data.