search

PSIAIMS / CAMIS

https://psiaims.github.io/CAMIS/
Apache License 2.0
55 stars 51 forks source link

Add Python binomial test #248

Closed thisisnic closed 5 days ago

thisisnic commented 2 months ago

Draft PR to add Python binomial test - I think we'll need to update requirements.txt as well.

We could also add guidance around how to update requirements.txt.

LucaAnholt commented 2 months ago

Looks like the build failure might be down to a scipy version issue.

In the requirments.txt scipy is at version 1.13.0, and from looking online it looks like the binom_test module was refactored in version 1.12.0.

from scipy.stats import binomtest should work in versions > 1.12.0

thisisnic commented 1 week ago

In the end I removed the additional dependency which was needed for the dataset and instead just used the CSV file containing the data which was added on a different PR.