Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

α → β Ⅳ #300

Merged
merged 19 commits into from
Jun 28, 2020
Merged

α → β Ⅳ #300

merged 19 commits into from
Jun 28, 2020

Conversation

HebaruSan
Copy link
Contributor

@HebaruSan HebaruSan commented Jun 28, 2020

Successor to #242 and #267 and #285.
Fixes and improvements, #299 is critical.

Features:

Performance fixes:

Bug fixes:

Internal:

HebaruSan and others added 19 commits June 13, 2020 17:24
Index columns used for order_by
This commit adds a celery background task triggered every 24 hours,
that goes through the entire mod table and calculates the mod scores and saves them in the new Mod.score column.
Mod.score is also updated whenever a value changes that affects the score (mod download, follow event, ...).

`search_mods` now simply orders the mods by this new column (after filtering by search terms and more) and applies an offset and limit.
This needs way less compute time than calculating the score of each mod on each request and paginating afterwards.
Precalculate mod scores in periodic celery background task
Give new foreign key in 7993492be4eb a name to make downgrades work
Don't require smtp-user and -password to be set in send_mail()
@HebaruSan HebaruSan merged commit c3351a9 into beta Jun 28, 2020
@HebaruSan HebaruSan mentioned this pull request Jun 28, 2020
This was referenced Oct 5, 2020
@HebaruSan HebaruSan mentioned this pull request Mar 14, 2021
@HebaruSan HebaruSan mentioned this pull request May 21, 2021
This was referenced Oct 2, 2021
@HebaruSan HebaruSan mentioned this pull request Apr 25, 2022
@HebaruSan HebaruSan mentioned this pull request Aug 31, 2022
@HebaruSan HebaruSan mentioned this pull request Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants