search

cta-observatory / cta-lstchain

LST prototype testbench chain
https://cta-observatory.github.io/cta-lstchain/
BSD 3-Clause "New" or "Revised" License
24 stars 77 forks source link

docs build fialing due to seaborn style in mc sensitivity #1161

Closed maxnoe closed 1 year ago

maxnoe commented 1 year ago 
 Failed to import lstchain.scripts.lstchain_mc_sensitivity.
Possible hints:
* OSError: 'seaborn-deep' is not a valid package style, path of style file, URL of style file, or library style name (library styles are listed in `style.available`)
* AttributeError: module 'lstchain.scripts' has no attribute 'lstchain_mc_sensitivity'
morcuended commented 1 year ago

Recent matplotlib release 3.8.0 (2023-09-13) dropped support for the seaborn-<foo> styles -> use seaborn-v0_8-<foo> instead, or directly use the seaborn API.

morcuended commented 1 year ago

I propose pinning matplotlib to ~=3.7.0 to avoid jumping to 3.8 for the time being. I think this anyway has to be fixed in ctaplot

rlopezcoto commented 1 year ago

I propose pinning matplotlib to ~=3.7.0 to avoid jumping to 3.8 for the time being. I think this anyway has to be fixed in ctaplot

sounds good as a temporary solution, let's in any case leave this open or open a new Issue to remember to unpin matplotlib whenever this is fixed in ctaplot (@vuillaut)

maxnoe commented 1 year ago

@vuillaut is fixing this in ctaplot, so we could also just update once ready.

maxnoe commented 1 year ago

https://github.com/cta-observatory/ctaplot/pull/201