A log_export option was added to the SpyglassMixin.fetch_nwb()
If a function call does not explicitly pass a value for log_export, the first vallue in attrs it passes (if any) is taken as the log_export argument, leading to errors later
So far this only caused an issue for me when getting data from DLCV1 which passes a list of attrs in its fetch_nwb call
Describe the bug
log_export
option was added to theSpyglassMixin.fetch_nwb()
log_export
, the first vallue inattrs
it passes (if any) is taken as thelog_export
argument, leading to errors laterDLCV1
which passes a list ofattrs
in itsfetch_nwb
callTo Reproduce
Solution Make
log_export
part of thekwargs
and assign a default value in the function