Skip to content

Commit

Permalink
Merge #234
Browse files Browse the repository at this point in the history
234: Pin down scipy version r=chrysn a=chrysn

Newer versions cause build failures because they expect a newer cython than is provided in jammy; they'd show errors like:

```
      Error compiling Cython file:
      ------------------------------------------------------------
      ...
      ctypedef float s                                                                                                                             ctypedef double d                                                                                                                            ctypedef float complex c
      ctypedef double complex z
      cdef void caxpy(int *n, c *ca, c *cx, int *incx, c *cy, int *incy) noexcept nogil
                                                                        ^
      ------------------------------------------------------------

      /usr/local/lib/python3.10/dist-packages/scipy/linalg/cython_blas.pxd:20:67: Syntax error in C variable declaration
```

Let's hope this allows building again -- if not, we have multiple bitrots to combat.

Co-authored-by: chrysn <[email protected]>
  • Loading branch information
bors[bot] and chrysn authored Oct 12, 2023
2 parents 3b18ad8 + e142eb6 commit 3137389
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions riotbuild/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ colorama>=0.4.0
cryptography==3.3.2
scapy>=2.4.3
protobuf==3.18.3
scipy == 1.10.1 # needed by scikit-learn, but newer ones fail with jammy's cython
scikit-learn==0.22.1
joblib==1.2.0
emlearn==0.10.1
Expand Down

0 comments on commit 3137389

Please sign in to comment.