Skip to content

Commit b19b9ca

Browse files
committed
fix adwin
1 parent 9d4de29 commit b19b9ca

File tree

4 files changed

+8
-49
lines changed

4 files changed

+8
-49
lines changed

CONTRIBUTING.md

+6-35
Original file line numberDiff line numberDiff line change
@@ -106,40 +106,10 @@ From the root of the repository, you can then run the `make livedoc` command to
106106

107107
All classes and function are automatically picked up and added to the documentation. The only thing you have to do is to add an entry to the relevant file in the [`docs/releases` directory](docs/releases).
108108

109-
## Building Cython extensions
110-
111-
```sh
112-
make build-cython
113-
```
114-
115-
## Building Rust extensions
116-
117-
Debug settings:
118-
119-
```sh
120-
make develop
121-
```
122-
123-
Release settings:
124-
125-
```sh
126-
make build-rust
127-
```
128-
129-
After building the project by modifying the rust part of the codebase (changing the project architecture, renaming it, etc.), it happens that by importing `river,` the python process is killed. If this happens, we invite you to remove the following things and start a new build:
130-
131-
```sh
132-
# remove all .so output from rust ie river/stats/_rust_stats.cpython*
133-
rm -rf target
134-
rm -rf river.egg-info
135-
rm Cargo.lock
136-
rm -rf build
137-
```
138-
139109
## Build Cython and Rust extensions
140110

141111
```sh
142-
make build_all
112+
poetry install
143113
```
144114

145115
## Testing
@@ -182,10 +152,11 @@ make execute-notebooks
182152
2. Run `make execute-notebooks` just to be safe
183153
3. Run the [benchmarks](benchmarks)
184154
4. Bump the version in `river/__version__.py`
185-
5. Tag and date the `docs/releases/unreleased.md` file
186-
6. Commit and push
187-
7. Wait for CI to [run the unit tests](https://github.com/online-ml/river/actions/workflows/ci.yml)
188-
8. Push the tag:
155+
5. Bump the version in `pyproject.toml`
156+
6. Tag and date the `docs/releases/unreleased.md` file
157+
7. Commit and push
158+
8. Wait for CI to [run the unit tests](https://github.com/online-ml/river/actions/workflows/ci.yml)
159+
9. Push the tag:
189160

190161
```sh
191162
RIVER_VERSION=$(python -c "import river; print(river.__version__)")

Makefile

-12
Original file line numberDiff line numberDiff line change
@@ -25,15 +25,3 @@ livedoc: doc
2525

2626
rebase:
2727
git fetch && git rebase origin/main
28-
29-
develop:
30-
python ./setup.py develop
31-
32-
build-cython:
33-
python setup.py build_ext --inplace --force
34-
35-
build-rust:
36-
python setup.py build_rust --inplace --release
37-
38-
build:
39-
python setup.py build_rust --inplace --release build_ext --inplace --force

pyproject.toml

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
44

55
[tool.poetry]
66
name = "river"
7-
version = "0.20.1"
7+
version = "0.21.0"
88
description = "Online machine learning in Python"
99
readme = "README.md"
1010
homepage = "https://riverml.xyz/"

river/drift/adwin_c.pyx

+1-1
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ cdef class AdaptiveWindowing:
8888
If True then a change is detected.
8989
9090
"""
91-
self._update(value)
91+
return self._update(value)
9292

9393
cdef bint _update(self, double value):
9494
# Increment window with one element

0 commit comments

Comments
 (0)