Sambit Panda
Sambit Panda
Agreed seems low priority to me
Not that I know of, but we can add a parameter when building the object
It does, but those are in main but haven't been released on PyPi. We are also currently working on conditional Dcorr. To use them now you would need to build...
Current module: hyppo/conditional Conditional dcorr pr: https://github.com/neurodata/hyppo/pull/349
Yep, we have fixed this issue in hyppo v0.5.2
@oakla PR will automatically close when issue is merged
Thanks for the contribution. Do you have unit tests showing that this change is right? That would involve ensuring that the change is the same as `pearsonr` in SciPy when...
Thank you for your contribution! Changes look good to me. Once the tests pass, I will merge
I'm happy to merge if @adam2392 is cool with it
@j1c The only thing missing is a few unit tests for some of the error checks. Once those are added, I'll merge