search

cosanlab / nltools

Python toolbox for analyzing imaging data
https://nltools.org
MIT License
120 stars 42 forks source link

Bug fix #414

Closed ljchang closed 1 year ago

ljchang commented 1 year ago

@ejolly I added a quick fix to issue #413. Basically, I just added a check to see if the variable exists. Seems to address the reported issue.

ejolly commented 1 year ago

Will take a look this week!

ejolly commented 1 year ago

@ljchang I ended up fixing this a bit more robustly, by making a copy of the dataframe at the beginning (which I should have been doing before). Also added this as a test case. I ended up pushing directly to master so I'm going to close this PR without merging if that's ok/