Skip to content

Fix/pop in cat refit#163

Merged
blondered merged 6 commits intomainfrom
fix/pop_in_cat_refit
Jul 16, 2024
Merged

Fix/pop in cat refit#163
blondered merged 6 commits intomainfrom
fix/pop_in_cat_refit

Conversation

@blondered
Copy link
Copy Markdown
Collaborator

@blondered blondered commented Jul 16, 2024

@blondered blondered requested a review from feldlime July 16, 2024 09:48
@blondered blondered self-assigned this Jul 16, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (9b3992e) to head (2ecedcb).
Report is 68 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##              main      #163     +/-   ##
===========================================
  Coverage   100.00%   100.00%             
===========================================
  Files           45        55     +10     
  Lines         2242      3304   +1062     
===========================================
+ Hits          2242      3304   +1062     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment thread rectools/models/popular_in_category.py Outdated
Comment thread tests/model_selection/test_cross_validate.py Outdated
Comment thread rectools/models/popular_in_category.py Outdated
@blondered blondered merged commit 374b4ae into main Jul 16, 2024
@blondered blondered deleted the fix/pop_in_cat_refit branch July 16, 2024 10:48
blondered added a commit that referenced this pull request Sep 23, 2024
- Fixed `PopularInCategoryModel` refit behaviour and `cross-validate`
compatibility
- Fixed `PopularInCategoryModel` empty category interactions behaviour
- Added tests
Closes #162
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PopularInCategoryModel fails to fit second time

2 participants