maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fast follow on follow suggestion efficiency (#20709)

Browse source
This commit is contained in:
Ben Halpern 2024-02-29 18:05:21 -05:00 • committed by GitHub
parent 446a93c3a5
commit 2ad8b7a809
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
app/queries/users/suggest_prominent.rb
View file

@ -26,14 +26,13 @@ module Users
def fetch_and_pluck_user_ids
filtered_articles = if tags_to_consider.any?
Article.cached_tagged_with_any(tags_to_consider)
Article.published.cached_tagged_with_any(tags_to_consider)
else
Article.featured
Article.published.featured
end
order = Arel.sql("(hotness_score * (feed_success_score - clickbait_score)) DESC")
user_ids = filtered_articles.order(order).limit(RETURNING * 2).pluck(:user_id) - [user.id]
user_ids = filtered_articles.order("hotness_score DESC").limit(RETURNING * 2).pluck(:user_id) - [user.id]
if user_ids.size > (RETURNING / 2)
user_ids
user_ids.sample(RETURNING)
else
# This is a fallback in case we don't have enough users to return
# Will generally not be called — but maybe for brand new forems