Skip to content

Commit

Permalink
Remove scipy version requirement (#251)
Browse files Browse the repository at this point in the history
Proper include structure removes the need, we hope.

Co-authored-by: Andrew Grant <[email protected]>
  • Loading branch information
gahtan-syarif and AndyGrant authored Nov 13, 2024
1 parent 01c2dda commit 66b8c24
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion OpenBench/stats.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@

import math
import scipy
from scipy import optimize

def TrinomialSPRT(results, elo0, elo1):

Expand Down Expand Up @@ -114,7 +115,7 @@ def secular(pdf):
def f(x):
return sum([pi * ai / (1 + x * ai) for ai, pi in pdf])

x, res = scipy.optimize.brentq(
x, res = optimize.brentq(
f, l + epsilon, u - epsilon, full_output=True, disp=False
)
assert res.converged
Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Django==4.2.1
django-htmlmin==0.11.0
requests
scipy==1.9.0
scipy

0 comments on commit 66b8c24

Please sign in to comment.