search

BU-hammerTeam / PyHammer

A Python Spectral Typing Suite
MIT License
53 stars 14 forks source link

setup matplotlib and pyplot #1

Closed jvntra closed 8 years ago

jvntra commented 8 years ago

was crashing on launch (via >python pyhammer.py) and this solution fixed: http://stackoverflow.com/questions/32019556/matplotlib-crashing-tkinter-application

zephyr5050 commented 8 years ago

That is a known issue at this point and a fix has already been implemented in eyecheck.py. However, this crash and fix is limited to specific environments and adding the suggested code will cause other computers/environments to crash. The decided approach is to allow matplotlib to use its default backend and only if that doesn't work, the user can edit the indicated lines of eyecheck.py to switch the backend.

ctheissen commented 8 years ago

I personally think this should be an internal check of the package to see if the user is on a Windows or Mac machine. This can be done through the 'os' package. You want to make this as easy as possible for the user.

zephyr5050 commented 8 years ago

Yes that is certainly true, and something I'm investigating. The only potential issue is that not all Mac users seem to face this problem and so implementing this fix for all Macs might not be appropriate.