Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some basic statistical methods? #8

Merged
merged 18 commits into from
Aug 21, 2023
Merged

Conversation

Quafadas
Copy link
Contributor

@Quafadas Quafadas commented Aug 3, 2023

Hi,

I ended up doing a little more than I thought at the outset. If you aren't keen on having statistical methods in here, that's fine you only need to say.

I added

  • sample mean
  • sample variance
  • sample std
  • covariance
  • Pearson correlation
  • spearman's rank (<- as discussed)

In order to believe I had these done reasonably accurately, I added unit tests, and configured the testing module to test in CI. I hope that's okay - it's the only way I could get native to run at all.

I also struggled to get native to compile on my machine without updating a bunch of dependancies, so I did that too. I hope that isn't problematic, I can revert them if so...

@dragonfly-ai dragonfly-ai merged commit ee2d24e into dragonfly-ai:main Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants