Skip to content

Commit

Permalink
Implicit version update (#45)
Browse files Browse the repository at this point in the history
- Bumped implicit version to 0.7.1
- Updated implicit models wrappers and tests
- Temporarily removed fitting_features_together=True support for ALS model
  • Loading branch information
blondered authored Oct 4, 2023
1 parent cd28c50 commit 0fe1bc0
Show file tree
Hide file tree
Showing 8 changed files with 272 additions and 401 deletions.
23 changes: 21 additions & 2 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,26 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0


## Unreleased

### Added
- Added `MRR (Mean Reciprocal Rank)` to `metrics` ([#29](https://github.com/MobileTeleSystems/RecTools/pull/29))
- Added `F1beta`, `MCC (Matthew correlation coefficient)` to `metrics` ([#32](https://github.com/MobileTeleSystems/RecTools/pull/32))
- Added `LastNSplitter` to `model_selection` ([#33](https://github.com/MobileTeleSystems/RecTools/pull/32))
- Added random `KFoldSplitter` to `model_selection` ([#31](https://github.com/MobileTeleSystems/RecTools/pull/31))
-

### Changed
- Bumped `implicit` version to 0.7.1 ([#45](https://github.com/MobileTeleSystems/RecTools/pull/45))
- Bumped `poetry` version to 1.4.0 for github workflows ([#43](https://github.com/MobileTeleSystems/RecTools/pull/43))
- Updated dependencies ([#43](https://github.com/MobileTeleSystems/RecTools/pull/43))
- Moved `nmslib` from main dependencies to extras ([#36](https://github.com/MobileTeleSystems/RecTools/pull/36))
- Added base `Splitter` class to construct data splitters ([#31](https://github.com/MobileTeleSystems/RecTools/pull/31))
- Updated notebooks in `examples` ([#44](https://github.com/MobileTeleSystems/RecTools/pull/44))

### Fixed
- Fixed bugs with new version of `pytorch_lightning` ([#43](https://github.com/MobileTeleSystems/RecTools/pull/43))
- Fixed `pylint` config for new version ([#43](https://github.com/MobileTeleSystems/RecTools/pull/43))
- Fixed CI ([#40](https://github.com/MobileTeleSystems/RecTools/pull/40)) ([#34](https://github.com/MobileTeleSystems/RecTools/pull/34))
- Fixed cyclic imports ([#45](https://github.com/MobileTeleSystems/RecTools/pull/45))

- Mean Reciprocal Rank metric
### Removed
- Temporarily removed support for fitting ALS model with features together ([#45](https://github.com/MobileTeleSystems/RecTools/pull/45))
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -82,11 +82,11 @@ To install all requirements run
```
make install
```
You must have `python3` and `poetry` installed.
You must have `python3` and `poetry==1.4.0` installed.

For autoformatting run
```
make autoformat
make format
```

For linters check run
Expand Down
219 changes: 133 additions & 86 deletions poetry.lock

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ numpy = ">=1.19.5, <2.0.0"
pandas = ">=0.25.3, <2.0.0"
scipy = "^1.5.4"
tqdm = "^4.27.0"
implicit = "0.4.4"
implicit = "^0.7.1"
attrs = ">=19.1.0,<22.0.0"
typeguard = "^2.0.1"
lightfm = "1.17"
Expand Down
2 changes: 1 addition & 1 deletion rectools/metrics/diversity.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,10 @@
import pandas as pd

from rectools import Columns
from rectools.metrics import PairwiseDistanceCalculator
from rectools.utils import select_by_type

from .base import MetricAtK
from .distances import PairwiseDistanceCalculator


@attr.s
Expand Down
Loading

0 comments on commit 0fe1bc0

Please sign in to comment.