Eric Larson
Eric Larson
Sorry we forgot about this one @szz-dvl -- feel free to ping us if it's time for a (re-)review and it has been a couple of weeks! I pushed a...
It has to do with CircleCI build permissions... I pushed an empty commit that should come back green. I'll mark for merge-when-green, thanks in advance @szz-dvl !
I've enabled `cibuildwheel` for several repos in the past (e.g., `openmeeg`), I'm happy to try giving it a shot here!
I like the improved logging idea
We should fix `plot_cov` to deal with diagonal noise covariances (which includes those returned by `make_ad_hoc_cov`)
For the hook I think it's easiest probably just to add an exception to the hook code for now. We've hit that problem before and I think that's how we...
The test probably needs to be adjusted for this case. It's meant to catch importing classes and functions (which we always want to make explicit like `from . import `,...
It does flake out from time to time
Sounds reasonable to me! One hiccup could be licensing -- if you want to look at their code and understand what to do their code needs to be in a...
Argh @sappelhoff I went to look at `pybv` and the code there seems to have to do almost exclusively with writing, not reading BV data. @sappelhoff WDYT about just keeping...