search

PESTools / pestools

PESTools
12 stars 10 forks source link

RES obs_info_file error #51

Closed pjuckem closed 9 years ago

pjuckem commented 9 years ago

I get an error when using pestools.Res without an obs_info_file. I had cloned the Barr repo of pestools, but uninstalled it (pip uninstall pestools), cloned the github repo, and installed that one, but still same problem. It runs fine if the obs_info_file for the CC model is supplied. Here are a couple of screen shots: image image . . . image

mnfienen commented 9 years ago

On my Mac I found a different error (just pull-requested a fix....it was minor) but then the code causing Paul trouble worked fine for me

pjuckem commented 9 years ago

It's probably just me....though I'm not sure what's been corrupted on my system. Will look into it more & try on my laptop which has Anaconda instead of Enthought... Paul

On Tue, Jun 23, 2015 at 8:48 AM, mnfienen notifications@github.com wrote:

On my Mac I found a different error (just pull-requested a fix....it was minor) but then the code causing Paul trouble worked fine for me

— Reply to this email directly or view it on GitHub https://github.com/PESTools/pestools/issues/51#issuecomment-114511761.

echristi commented 9 years ago

I'm able to reproduce Paul's issue. Haven't had a chance to dig into why.

I didn't see Mike's pull request.

mnfienen commented 9 years ago

I see Andy fixed the bug I was going to pull request about. I was working on an old fork which now I updated. Still can't reproduce Paul's error though...

echristi commented 9 years ago

I hadn't updated since Andy made changes either. Once I did that and re-ran setup.py I no longer get the error Paul was getting.

pjuckem commented 9 years ago

ARG! Thanks for the gentle reminder on setup.py. I had checked "git log" to ensure Andy's fix had been pulled, but hopefully this will burn "git pull", then "python setup.py install" into my memory! All is well on both windows machines. Thanks. I'll close this issue.