# Summarise This Topic not working for parameters smaller than 50

**URL:** https://meta.discourse.org/t/summarise-this-topic-not-working-for-parameters-smaller-than-50/147185
**Category:** Support
**Created:** [April 7, 2020, 8:51pm UTC](https://meta.discourse.org/t/summarise-this-topic-not-working-for-parameters-smaller-than-50/147185 "2020-04-07T20:51:10Z")
**Posts on this page:** 8
**Page:** 1

<div class="post-metadata">

### Author: ![Shubham\_Ranjan](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/shubham_ranjan/32/156647_2.png) [@Shubham\_Ranjan](https://meta.discourse.org/u/Shubham_Ranjan)
#### Post date: [April 7, 2020, 8:51pm UTC](https://meta.discourse.org/t/summarise-this-topic-not-working-for-parameters-smaller-than-50/147185/1 "2020-04-07T20:51:10Z")

</div>

Hi,

I want to summarise topics with more than 20 posts. I changed this value `summary posts required` in settings to 20 even after that the `Summarize This Topic` button does not appear.

However, it’s working fine for the value **50**.

Please let me know if I’m doing something wrong.

Thanks!

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [April 7, 2020, 10:37pm UTC](https://meta.discourse.org/t/summarise-this-topic-not-working-for-parameters-smaller-than-50/147185/2 "2020-04-07T22:37:50Z")

</div>

Can we repro this @tshenry?

---

<div class="post-metadata">

### Author: ![tshenry](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/tshenry/32/119495_2.png) [@tshenry](https://meta.discourse.org/u/tshenry)
#### Post date: [April 7, 2020, 11:37pm UTC](https://meta.discourse.org/t/summarise-this-topic-not-working-for-parameters-smaller-than-50/147185/4 "2020-04-07T23:37:31Z")

</div>

I’m getting a weird déjà vu here, but can’t find any previous discussion. In any case, my understanding is that changing this setting does not result in an instantaneous retroactive change.

The Summarize This Topic button will be added to all topics that meet the criteria as a result of a scheduled job. It looks like for recent topics, it should update them within 15 minutes of changing the setting. For older topics, within a week. If anyone wants to make the change right away, it appears triggering a rebuild will do the trick.

Also good to keep in mind the `summary likes required` site setting will also control whether or not a topic is summarized.

Can you let me know if you are seeing anything contrary to the above @Shubham_Ranjan?

@codinghorror, do you think it might be worth adding a note to the setting that informs admins that it may take up to a week for all posts to be updated?

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [April 8, 2020, 12:23am UTC](https://meta.discourse.org/t/summarise-this-topic-not-working-for-parameters-smaller-than-50/147185/5 "2020-04-08T00:23:54Z")

</div>

> [@tshenry](#):
>
> do you think it might be worth adding a note to the setting that informs admins that it may take up to a week for all posts to be updated?

Oh yes definitely, I wasn’t even aware! Will the topics actually get updated? I’m unclear what the “update” process even is in this case.

---

<div class="post-metadata">

### Author: ![tshenry](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/tshenry/32/119495_2.png) [@tshenry](https://meta.discourse.org/u/tshenry)
#### Post date: [April 8, 2020, 12:49am UTC](https://meta.discourse.org/t/summarise-this-topic-not-working-for-parameters-smaller-than-50/147185/6 "2020-04-08T00:49:16Z")

</div>

Assuming I correctly followed all off the twists and turns in the code, here’s what I found:

We have a `ScoreCalculator` class that uses the two settings (`summary likes required` and `summary posts required`) to determine whether the topic should be summarized:

> <https://github.com/discourse/discourse/blob/main/lib/score_calculator.rb#L88-L89>

That `ScoreCalulator` class will be called in the `PeriodicalUpdates` job that runs every 15 minutes to check against topics that meet the specified arguments:

> <https://github.com/discourse/discourse/blob/main/app/jobs/scheduled/periodical_updates.rb#L22-L25>

All other topics should be covered by the `Weekly` job with:

[https://github.com/discourse/discourse/blob/master/app/jobs/scheduled/weekly.rb#L11](https://github.com/discourse/discourse/blob/master/app/jobs/scheduled/weekly.rb#L11)

Any particular preference for the copy of the setting description? Maybe something like:

 ![Screen Shot 2020-04-07 at 5.47.08 PM](https://global.discourse-cdn.com/meta/original/3X/e/9/e99a538a3a564ab7f24c999e854d5a3364822435.png)

I guess another alternative would be to run the score calculation upon saving the setting.

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [April 8, 2020, 1:00am UTC](https://meta.discourse.org/t/summarise-this-topic-not-working-for-parameters-smaller-than-50/147185/7 "2020-04-08T01:00:39Z")

</div>

Sure go for that copy! Looks good. We’ve definitely hit rule of three on this one. I didn’t even know!

---

<div class="post-metadata">

### Author: ![tshenry](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/tshenry/32/119495_2.png) [@tshenry](https://meta.discourse.org/u/tshenry)
#### Post date: [April 10, 2020, 4:50pm UTC](https://meta.discourse.org/t/summarise-this-topic-not-working-for-parameters-smaller-than-50/147185/9 "2020-04-10T16:50:05Z")

</div>

Done!

[https://github.com/discourse/discourse/commit/617692a6c43a21b0c158f3cea53f1142b4a35195](https://github.com/discourse/discourse/commit/617692a6c43a21b0c158f3cea53f1142b4a35195)

---

<div class="post-metadata">

### Author: ![tshenry](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/tshenry/32/119495_2.png) [@tshenry](https://meta.discourse.org/u/tshenry)
#### Post date: [April 16, 2020, 1:00am UTC](https://meta.discourse.org/t/summarise-this-topic-not-working-for-parameters-smaller-than-50/147185/10 "2020-04-16T01:00:04Z")

</div>

This topic was automatically closed after 5 days. New replies are no longer allowed.
