paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-18 12:54:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

digest: Create MAX_HOT_TOPICS_TO_BE_INCLUDED_IN_DIGEST constant.

Browse Source
This commit is contained in:
Vishnu KS
2021-02-09 00:27:23 +05:30
committed by Tim Abbott
parent c0bd05b52d
commit 738d759e6f
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
zerver/lib/digest.py
View File

@@ -31,6 +31,7 @@ logger = logging.getLogger(__name__)
log_to_file(logger, settings.DIGEST_LOG_PATH) log_to_file(logger, settings.DIGEST_LOG_PATH)
DIGEST_CUTOFF = 5 DIGEST_CUTOFF = 5
MAX_HOT_TOPICS_TO_BE_INCLUDED_IN_DIGEST = 4
TopicKey = Tuple[int, str] TopicKey = Tuple[int, str]
@@ -190,12 +191,12 @@ def get_hot_topics(
# Start with the two most diverse topics. # Start with the two most diverse topics.
hot_topics = topics_by_diversity[:2] hot_topics = topics_by_diversity[:2]
# Pad out our list up to 4 items, using the topics' length (aka message # Pad out our list up to MAX_HOT_TOPICS_TO_BE_INCLUDED_IN_DIGEST items,
# count) as the secondary filter. # using the topics' length (aka message count) as the secondary filter.
for topic in topics_by_length: for topic in topics_by_length:
if topic not in hot_topics: if topic not in hot_topics:
hot_topics.append(topic) hot_topics.append(topic)
if len(hot_topics) == 4: if len(hot_topics) == MAX_HOT_TOPICS_TO_BE_INCLUDED_IN_DIGEST:
break break
return hot_topics return hot_topics