docs: clarify FeatureSketch density and update model#19
Merged
Conversation
Owner
Bing-su
commented
May 29, 2026
Bing-su
commented
- restructure density, update-order, and adaptation notes with tables and diagrams
- make count-min density wording explicit as an estimate
- document score-before-update flow and lazy decay behavior
- correct FeatureSketch seed default to random
- restructure density, update-order, and adaptation notes with tables and diagrams - make count-min density wording explicit as an estimate - document score-before-update flow and lazy decay behavior - correct FeatureSketch seed default to random
There was a problem hiding this comment.
Pull request overview
Documentation-only update to docs/featuresketch/research.md clarifying FeatureSketch's density model, scoring/update semantics, lazy decay, and adaptation behavior, plus correcting the default seed value.
Changes:
- Restructure density, update-order, and adaptation sections with tables and mermaid diagrams; clarify count-min density as an estimate.
- Document
score/update/update_and_scoreflow explicitly as score-before-update with lazy decay. - Correct the default seed in the configuration table from
buildertorandom.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.