Open ianthomas23 opened 1 year ago
Sounds very promising. The tornado errors that I've encountered recently have appeared to be due to zero-length messages implying that Datashader output wasn't available when expected, generally with the symptom that the interactive plots fail to update on zoom or pan. If what you see is similar, that's definitely something to address before merge, as the interactive updates are critical for this application, and definitely test the interactive capabilities when debugging.
Work in progress to update the census example environment to use the latest
datashader
,bokeh
,panel
, etc. Initially usingdatashader
dev release0.14.5a
frompyviz/label/dev
channel, and other packages fromconda-forge
rather thandefaults
until everything required is ondefaults
, which is well underway. The other change is to usepyarrow
rather thanfastparquet
.It results in the following packages used:
It still includes the workaround for https://github.com/holoviz/datashader/issues/1202.
Tested locally on
osx-arm64
, historically one of the most problematic. It gives a bunch ofdatashader
warnings, all of which are fixed and will be in the imminent0.15.0
release. The output renders correctly for me except one plot is missing with atornado
error reported at the command-line, and the interactive plots at the end work correctly.Given these results, I conclude that there is nothing fundamentally wrong with
datashader
to prevent going ahead with the0.15.0
release. Once that is done here is a possible plan:datashader 0.15.0
to confirm that warnings have gone.holoviz
packages are available ondefaults
.defaults
rather thanconda-forge
andpyviz/label/dev
.tornado
error if it still occurs.