HomePhorge

Parameterize the common ngrams threshold

Description

Parameterize the common ngrams threshold

Summary:
Ref T13000. Since other changes have generally made the ngrams table manageable, I'm not planning to enable common ngrams by default at this time.

Instead, make the threshold configurable with "--threshold" so we can guide installs through tuning this if they want (e.g. PHI110), and tune hosted instances.

(This might eventually become automatic, but just smoothing this bit off for now feels reasonable to me.)

Test Plan: Ran with --reset, and with various invalid and valid --threshold arguments.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13000

Differential Revision: https://secure.phabricator.com/D18710

Details

Provenance
epriestleyAuthored on Oct 17 2017, 1:58 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP819b833607a6: Tweak rate limiting point counts for omnipotent users
Branches
Unknown
Tags
Unknown

Event Timeline