search

holoviz / datashader

Quickly and accurately render even the largest data.
http://datashader.org
BSD 3-Clause "New" or "Revised" License
3.24k stars 363 forks source link

Fix test suite #1314

Closed hoxbro closed 4 months ago

hoxbro commented 5 months ago

This fixes the failing tests and removes warnings from the test CI.

Also fixes a problem with Windows not solving the conda environment.

Before this PR: 13 failed, 1581 passed, 62 skipped, 10 xfailed, 13569 warnings After this PR: 1581 passed, 65 skipped, 11 xfailed

hoxbro commented 5 months ago

Still seems to be problem solving on Windows:

  Install runtime dependencies with: conda install -y   "colorcet" "dask-core" "multipledispatch" "numba" "numpy" "pandas" "param" "pillow" "pyct" "requests" "scipy" "toolz" "xarray" "dask-geopandas" "geopandas" "shapely >=2.0.0" "codecov" "geodatasets" "fastparquet" "flake8" "nbconvert" "nbformat" "nbsmoke >0.5" "netcdf4" "pyarrow" "pytest <8" "pytest-benchmark" "pytest-cov" "rasterio" "rioxarray" "scikit-image" "spatialpandas" "dask-geopandas" "geopandas" "shapely >=2.0.0" "bokeh >3.1" "geodatasets" "holoviews" "matplotlib-base >=3.3" "panel >1.1" "scikit-image" "spatialpandas"
  Install runtime dependencies with: conda install -y   "colorcet" "dask-core" "multipledispatch" "numba" "numpy" "pandas" "param" "pillow" "pyct" "requests" "scipy" "toolz" "xarray" "dask-geopandas" "geopandas" "shapely >=2.0.0" "codecov" "geodatasets" "fastparquet" "flake8" "nbconvert" "nbformat" "nbsmoke >0.5" "netcdf4" "pyarrow" "pytest <8" "pytest-benchmark" "pytest-cov" "rasterio" "rioxarray" "scikit-image" "spatialpandas" "dask-geopandas" "geopandas" "shapely >=2.0.0" "bokeh >3.1" "geodatasets" "holoviews" "matplotlib-base >=3.3" "panel >1.1" "scikit-image" "spatialpandas"
  Channels:
   - pyviz/label/dev
   - numba
   - conda-forge
  Platform: win-64
  Collecting package metadata (repodata.json): ...working... done
  warning  libmamba Added empty dependency for problem type SOLVER_RULE_UPDATE

Took 1.5 hours to solve for one of them....

codecov[bot] commented 5 months ago

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (662a0d6) 85.65% compared to head (8d3ada0) 85.64%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1314 +/- ## ========================================== - Coverage 85.65% 85.64% -0.02% ========================================== Files 52 52 Lines 11288 11291 +3 ========================================== + Hits 9669 9670 +1 - Misses 1619 1621 +2 ```
hoxbro commented 5 months ago

Failing tests on Linux are the same reason as https://github.com/holoviz/geoviews/pull/703, will solve it self (likely by the end of the day)

hoxbro commented 5 months ago

The slow solve Windows seems to be fixed by removing fastparquet. I don't think it is a culprit by itself, but it is likely caused by some combination with other packages.

hoxbro commented 5 months ago

Closed #1308, because numba 0.59 has been released and passed the CI as shown here

image