Skip to content

Auto-tune and expose social graph PR-rate thresholds#27

Open
jsholmes wants to merge 2 commits intomainfrom
feat/auto-tune-pr-rate-thresholds
Open

Auto-tune and expose social graph PR-rate thresholds#27
jsholmes wants to merge 2 commits intomainfrom
feat/auto-tune-pr-rate-thresholds

Conversation

@jsholmes
Copy link
Copy Markdown
Contributor

@jsholmes jsholmes commented Apr 8, 2026

Summary

  • auto-tune the social-graph PR-rate filter downward when the default threshold over-filters the graph
  • add --min-pr-rate for strict overrides and --min-active-author-fraction to control the retention floor
  • persist the chosen threshold in build metadata, surface it in status, and make export use cached built edges

Details

  • explicit --min-pr-rate now stays strict, warning on low retention and only failing when it produces zero edges
  • auto-tuning operates on the same recent-window candidate PR set used by the social graph itself
  • the PR-rate semantics are documented in code so the recent-window behavior is intentional and clear

Testing

  • ./.venv/bin/pytest -q
  • ./.venv/bin/ruff check src tests

jsholmes added 2 commits April 8, 2026 14:11
Step the PR-rate threshold down from the default when it would
over-filter the graph. Add --min-pr-rate for strict overrides and
--min-active-author-fraction to control the retention floor. Persist
the chosen threshold in build metadata, surface it in status, and make
export use cached built edges instead of recomputing with hidden
defaults.
@jsholmes jsholmes requested review from baell and pwerry as code owners April 8, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant