-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Create new feed-variant 20220422 #17406
Conversation
"matching_negative_tags_intersection_count": { | ||
"cases": [ | ||
[0, 1], | ||
[1, 0.8], | ||
[2, 0.5], | ||
[3, 0.3], | ||
[4, 0.2] | ||
], | ||
"fallback": 0 | ||
}, | ||
"matching_positive_tags_intersection_count": { | ||
"cases": [ | ||
[0, 0.7], | ||
[1, 0.85], | ||
[2, 0.93], | ||
[3, 0.98], | ||
[4, 1] | ||
], | ||
"fallback": 1 | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New change here
@@ -189,7 +189,7 @@ def self.lever_catalog | |||
AND comments.deleted = false | |||
AND comments.created_at > :oldest_published_at"]) | |||
|
|||
relevancy_lever(:matching_negative_tag_intersection_count, | |||
relevancy_lever(:matching_negative_tags_intersection_count, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch on consistent naming!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It may look like I want to request changes, but really I don't want to accidentally merge this. @JennieOcken wants this to go live on Monday.
For those reading along, we don't need tests because in spec/services/articles/feeds/variant_query_spec.rb we test all variants in that directory. |
What type of PR is this? (check all applicable)
Description
Add a new feed-variant as per Jennie's request. Shoutout to @jeremyf and @msarit
Related Tickets & Documents
n/a
QA Instructions, Screenshots, Recordings
n/a
Added tests?
Added to documentation?
[optional] Are there any post deployment tasks we need to perform?
n/a