You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Dec 19, 2023. It is now read-only.
Currently, we get all the submissions for originality to compare to with this query:
query = """
SELECT s.id FROM submissions s
INNER JOIN originalities o
ON s.id = o.submission_id
WHERE s.round_id = %s AND
s.user_id != %s AND
s.inserted_at < %s AND
s.selected = TRUE AND
(o.value = TRUE OR o.pending = TRUE)
ORDER BY s.inserted_at DESC"""
This is incorrect because of the selected = TRUE. This query would only be correct if it were run at the exact moment the submission were made, which is rarely the case because it's waiting in a queue. As users make new submissions, their old submissions will no longer have selected = TRUE, even though they did have selected = TRUE at the time my submission was made.
Essentially, originality will be easier to achieve the longer the submission waits in the queue because the number of submissions that are both older than mine and selected = TRUE can only decrease as the same users make more submissions.
The text was updated successfully, but these errors were encountered:
Currently, we get all the submissions for originality to compare to with this query:
This is incorrect because of the
selected = TRUE
. This query would only be correct if it were run at the exact moment the submission were made, which is rarely the case because it's waiting in a queue. As users make new submissions, their old submissions will no longer haveselected = TRUE
, even though they did haveselected = TRUE
at the time my submission was made.Essentially, originality will be easier to achieve the longer the submission waits in the queue because the number of submissions that are both older than mine and
selected = TRUE
can only decrease as the same users make more submissions.The text was updated successfully, but these errors were encountered: