search

maccallumlab / meld

Modeling with limited data
http://meldmd.org
Other
54 stars 28 forks source link

tkinter might not be shipped with python #159

Open ccccclw opened 1 year ago

ccccclw commented 1 year ago

This PR handles tkinter import error when it's not shipped with python installed on certain OS. In fact, I am not sure if we need this module or not here .

jlmaccal commented 1 year ago

I'm super confused here...

  1. Why do we need tkinter in the guts of MELD?
  2. Why are we importing E? It's not even used.
ccccclw commented 1 year ago

well..that's what I got confused too. Looks like it was somehow added around this point. I just removed the import.

ccccclw commented 1 year ago

Add another commit for resolving bugs in extract_trajectory.