cxcsds / ciao-contrib

Extra scripts and code to enhance the capabilities of CIAO.
GNU General Public License v3.0
8 stars 6 forks source link

scripts that plot need update for headless systems #746

Open kglotfelty opened 1 year ago

kglotfelty commented 1 year ago

With matplotlib 3.6.x, the TkAgg backend is now raising an exception if there is no DISPLAY rather than simply falling back to use the headless Agg backend. The scripts that do plotting should be updated to handle this more gracefully

DougBurke commented 1 year ago

That's nice of the backend.

DougBurke commented 1 year ago

@nplee - does the lightcurve stuff need to be updated for this? I'm less bothered with the sherpa_contrib stuff as I believe, but have not recently checked, that it's primarily for interactive use (although it can still be used in a headless manner, so preferably it should work).

nplee commented 1 year ago

I not sure if it's a terrible priority, since lc_clean/lc_sigma_clip have a plot argument to avoid any sort of error, but it's a simple enough update (#767)