search

neurodata / hyppo

Python package for multivariate hypothesis testing
https://hyppo.neurodata.io/
Other
215 stars 90 forks source link

[Issue 341] Updated energy stats calculation, aligned with dcor package results, and added unit test #367

Closed YuHuang0525 closed 1 year ago

YuHuang0525 commented 1 year ago

…ge results, and added unit test

Reference issue

Made changes to resolve Issue-341

Type of change

Updated ksample/energy.py file by changing the energy stats calculation, following along the paper theorem-2. Added unit test.

What does this implement/fix?

After the change, the energy statistics result from ksample follows along with paper calculation and dcor package's energy distance calculation.

Additional information

Please only consider the last commit and ignore the previous two as they are redundant. Thank you.

sampan501 commented 1 year ago

@YuHuang0525 see #368

sampan501 commented 1 year ago

Fixed in #368

YuHuang0525 commented 1 year ago

Thank you so much for your detailed reply, this is my first time trying to contribute to an open-source project. Really appreciate your time!