Change repost action to align with twitter Recsys
#31
Conversation
| @@ -428,8 +429,8 @@ async def repost(self, agent_id: int, post_id: int): | |||
| } | |||
|
|
|||
| post_type_result = self.pl_utils._get_post_type(post_id) | |||
| post_insert_query = ("INSERT INTO post (user_id, original_post_id)" | |||
| "VALUES (?, ?)") | |||
| post_insert_query = ("INSERT INTO post (user_id, original_post_id" | |||
There was a problem hiding this comment.
OK, the current_time is added, what would post["content"] be ? "None"?
| @@ -4,7 +4,7 @@ CREATE TABLE post ( | |||
| post_id INTEGER PRIMARY KEY AUTOINCREMENT, | |||
| user_id INTEGER, | |||
| original_post_id INTEGER, -- NULL if this is an original post | |||
| content TEXT, | |||
| content TEXT DEFAULT '', -- DEFAULT '' for initial posts | |||
There was a problem hiding this comment.
@Redtides0 I changed the default value of content as a empty string. I think this might be a more convenient way to implementat it? WDYT?
In the current main branch code, there is already a discussion on the classification of post types, so I did not modify the |
Ahh, my apologies for the initial oversight. After a more thorough review, I can confirm that everything is now in order. |
After discussing with @Redtides0 , I modified some details of the
repostaction implementation and default field in database to make it more compatible with Twitter's recommendation algorithm.