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.

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

Large gap between components of EEG viewer #64

Open
cbrnr opened this issue Jul 20, 2023 · 2 comments · Fixed by #65
Open

Large gap between components of EEG viewer #64

cbrnr opened this issue Jul 20, 2023 · 2 comments · Fixed by #65

Comments

@cbrnr
Copy link

cbrnr commented Jul 20, 2023

The gap (vertical distance) between the two components of the EEG viewer (signal browser and overview bar) is too large on my machine:

Screenshot 2023-07-20 at 12 33 54

I started the viewer with panel serve workflow_eeg-viewer.ipynb --show. Here's the output of import holoviews as hv; hv.extension("bokeh"); hv.show_versions():

Python              :  3.9.16 | packaged by conda-forge | (main, Feb  1 2023, 21:38:11) 
[Clang 14.0.6 ]
Operating system    :  macOS-13.4.1-arm64-arm-64bit
holoviews           :  1.16.2

IPython             :  8.14.0
bokeh               :  3.2.0
colorcet            :  3.0.1
cudf                :  -
dask                :  2023.7.0
datashader          :  0.15.1
geoviews            :  -
hvplot              :  0.8.4
ibis                :  -
jupyter_bokeh       :  -
jupyterlab          :  4.0.3
matplotlib          :  3.7.2
networkx            :  3.1
notebook            :  7.0.0
numba               :  0.57.1
numpy               :  1.24.4
pandas              :  2.0.3
panel               :  1.2.0
param               :  1.13.0
pillow              :  -
plotly              :  -
pyarrow             :  -
scipy               :  1.11.1
skimage             :  0.21.0
spatialpandas       :  -
streamz             :  -
xarray              :  2023.7.0
@droumis
Copy link
Collaborator

droumis commented Jul 25, 2023

Thanks! I've now updated this workflow to be wrapped in a set of Panel panes in such a way as to make the sizing of the line plot (eeg_viewer) extend to fill the available space, with a minimum height constraint.

Steps:

  • set aspect=1.5, responsive=True for the eeg_viewer
  • set height=100, responsive=True for the minimap
  • use pn.Column(pn.Row(eeg_viewer, min_height=500), minimap) for the layout

image

@droumis
Copy link
Collaborator

droumis commented Aug 8, 2023

reopened by #75 .. the rangetool issue was a bigger concerns. Will work on spacing another way

@droumis droumis reopened this Aug 8, 2023
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 a pull request may close this issue.

2 participants