re-added old commit from @awctomlinson that fixed pandas indexing error

LBHB / NEMS0

THIS VERSION OF NEMS IS NO LONGER SUPPORTED. PLEASE USE THE NEW NEMS REPOSITORY OR INSTALL NEMS_DB TO GET NEMS0 SUPPORT.

GNU General Public License v3.0

8 stars 4 forks source link

re-added old commit from @awctomlinson that fixed pandas indexing error #246

Closed jacobpennington closed 3 years ago

jacobpennington commented 3 years ago

It looks like this commit from @awctomlinson , which was present on the svdconv2d branch, somehow got left out in the merge. This is needed to avoid a pandas indexing when running tensorflow models. I tested the changes on a few models using this commit + the updated master branch and encountered no errors.

crheller commented 3 years ago

I noticed this when you were getting that indexing error recently, @jacobpennington . I agree, I think these commits should be merged into master, but I don't really use this code. Maybe double check with @svdavid before merging?