msmbuilder / msmbuilder-legacy

Legacy release of MSMBuilder
http://msmbuilder.org
GNU General Public License v2.0
25 stars 28 forks source link

bugfix in SampleTrajectory.py #435

Closed schwancr closed 9 years ago

schwancr commented 9 years ago

I thought I was in my fork, but apparently I wasn't... Anyway, it's a simple fix for #433

kyleabeauchamp commented 9 years ago

LGTM merge away if tests pass

schwancr commented 9 years ago

Apparently we've been failing for a while... I'm looking into it

kyleabeauchamp commented 9 years ago

Hmm, not sure what the deal is with the ACF there. IMHO not strictly necessary for you to solve that now, as it's clearly not related to this...

schwancr commented 9 years ago

Yea, I'll merge this and open a new PR.

It has to do with statsmodels I think. This test is using it.