[PR #154] [MERGED] 66 implement basic control over pool playback randomization #297

New issue

Closed

opened 2026-02-27 19:17:40 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 19:17:40 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/EddieTheCubeHead/Stagnum/pull/154
Author: @EddieTheCubeHead
Created: 3/30/2024
Status: ✅ Merged
Merged: 4/1/2024
Merged by: @EddieTheCubeHead

Base: master ← Head: 66-implement-basic-control-over-pool-playback-randomization

📝 Commits (10+)

cc934fd 66 Update database models to include required data
6865e76 66 Work towards randomization algorithm
052f730 66 Pool member weighted randomization works. Now onto user weighted randomization
b7316c6 66 More tests, small refactorings
3e4a5c8 66 Technical base for user playtime tracking
13bb8c4 66 User based randomization works
94059cb Merge branch 'master' into 66-implement-basic-control-over-pool-playback-randomization
af756f7 66 Register custom marker so CI/CD passes
c82d5ab Merge branch '66-implement-basic-control-over-pool-playback-randomization' of github.com:EddieTheCubeHead/Poolify into 66-implement-basic-control-over-pool-playback-randomization
a1226db 66 Shorten test runtime

📊 Changes

14 files changed (+580 additions, -51 deletions)

View changed files

📝 example.env (+7 -1)
📝 server/README.md (+17 -4)
📝 server/pytest.ini (+2 -1)
➕ server/src/alembic/versions/25c34e5bf930_data_changes_required_for_pool_.py (+38 -0)
📝 server/src/api/pool/dependencies.py (+62 -13)
➕ server/src/api/pool/randomization_algorithms.py (+166 -0)
📝 server/src/api/pool/tasks.py (+4 -3)
📝 server/src/database/entities.py (+20 -2)
📝 server/test/conftest.py (+37 -7)
📝 server/test/pool_features/conftest.py (+38 -12)
📝 server/test/pool_features/delete_content_features.py (+1 -0)
📝 server/test/pool_features/playback_features.py (+5 -7)
📝 server/test/pool_features/shared_pool_features.py (+4 -1)
➕ server/test/pool_features/weighted_randomization_features.py (+179 -0)

📄 Description

Most of the changes are in randomization_algorithm.py

Everything tested

~~User randomization is really agressive in the beginning and scales back quickly. Need to figure out something less agressive / more balanced in the future, but works for now.~~ edit: a less agressive algorithm has already been implemented on the PR

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/EddieTheCubeHead/Stagnum/pull/154 **Author:** [@EddieTheCubeHead](https://github.com/EddieTheCubeHead) **Created:** 3/30/2024 **Status:** ✅ Merged **Merged:** 4/1/2024 **Merged by:** [@EddieTheCubeHead](https://github.com/EddieTheCubeHead) **Base:** `master` ← **Head:** `66-implement-basic-control-over-pool-playback-randomization` --- ### 📝 Commits (10+) - [`cc934fd`](https://github.com/EddieTheCubeHead/Stagnum/commit/cc934fd53077903ac2bc035a6d6fd9b85fed8624) 66 Update database models to include required data - [`6865e76`](https://github.com/EddieTheCubeHead/Stagnum/commit/6865e767c122bc443d285ead150e9bc06e4371bd) 66 Work towards randomization algorithm - [`052f730`](https://github.com/EddieTheCubeHead/Stagnum/commit/052f730968accef515efaea09653db004b1be01e) 66 Pool member weighted randomization works. Now onto user weighted randomization - [`b7316c6`](https://github.com/EddieTheCubeHead/Stagnum/commit/b7316c6273f108eba887ae05ea73863cdaa21af5) 66 More tests, small refactorings - [`3e4a5c8`](https://github.com/EddieTheCubeHead/Stagnum/commit/3e4a5c8c584dc742fbffe1a6bac1c0170d772da1) 66 Technical base for user playtime tracking - [`13bb8c4`](https://github.com/EddieTheCubeHead/Stagnum/commit/13bb8c49e0ba4884936b636ff286f4cba6cc4f44) 66 User based randomization works - [`94059cb`](https://github.com/EddieTheCubeHead/Stagnum/commit/94059cb4043710dbf1ddd6d2ad7c20fb2cf30a38) Merge branch 'master' into 66-implement-basic-control-over-pool-playback-randomization - [`af756f7`](https://github.com/EddieTheCubeHead/Stagnum/commit/af756f7691db9fdd021cecc8f844209f01c7b7ec) 66 Register custom marker so CI/CD passes - [`c82d5ab`](https://github.com/EddieTheCubeHead/Stagnum/commit/c82d5abfac4c0d0483a0d64cd8308595d8bd431e) Merge branch '66-implement-basic-control-over-pool-playback-randomization' of github.com:EddieTheCubeHead/Poolify into 66-implement-basic-control-over-pool-playback-randomization - [`a1226db`](https://github.com/EddieTheCubeHead/Stagnum/commit/a1226dbb2c8906114b6ce5235e9d19d12d4fa141) 66 Shorten test runtime ### 📊 Changes **14 files changed** (+580 additions, -51 deletions) <details> <summary>View changed files</summary> 📝 `example.env` (+7 -1) 📝 `server/README.md` (+17 -4) 📝 `server/pytest.ini` (+2 -1) ➕ `server/src/alembic/versions/25c34e5bf930_data_changes_required_for_pool_.py` (+38 -0) 📝 `server/src/api/pool/dependencies.py` (+62 -13) ➕ `server/src/api/pool/randomization_algorithms.py` (+166 -0) 📝 `server/src/api/pool/tasks.py` (+4 -3) 📝 `server/src/database/entities.py` (+20 -2) 📝 `server/test/conftest.py` (+37 -7) 📝 `server/test/pool_features/conftest.py` (+38 -12) 📝 `server/test/pool_features/delete_content_features.py` (+1 -0) 📝 `server/test/pool_features/playback_features.py` (+5 -7) 📝 `server/test/pool_features/shared_pool_features.py` (+4 -1) ➕ `server/test/pool_features/weighted_randomization_features.py` (+179 -0) </details> ### 📄 Description Most of the changes are in randomization_algorithm.py Everything tested ~~User randomization is really agressive in the beginning and scales back quickly. Need to figure out something less agressive / more balanced in the future, but works for now.~~ edit: a less agressive algorithm has already been implemented on the PR --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>