search

mzechmeister / serval

calculate radial velocities from stellar spectra
MIT License
36 stars 9 forks source link

Matplotlib and tkinter required #68

Open phoenixsite opened 3 months ago

phoenixsite commented 3 months ago

Hello. I've just installed the program and tried the proposed example. Everything worked good but I noticed that matplotlib and tkinter are necessary to run srv. It may be useful including them in the requirements section of the README or, even better, having a requirements.txt file.

Good work! And thank you.

mzechmeister commented 3 months ago

Indeed, the -x option of srv uses matplotlib https://github.com/mzechmeister/serval/blob/b7c7b568ef3bb29e072d0cc6b25566ef8eb1a318/src/srv.py#L684 It's because of the sharex feature (gnuplot has unfortunately no analogue).

The same script also uses the gls. Matplotlib is usually often already installed. So I presume you encountered issues because of tkinter or because of an installation in some virtual environment.