Advanced aggregation options and built in per‑text explainability for rare‑class affinity#7
Open
ch1kim0n1 wants to merge 1 commit into
Open
Advanced aggregation options and built in per‑text explainability for rare‑class affinity#7ch1kim0n1 wants to merge 1 commit into
ch1kim0n1 wants to merge 1 commit into
Conversation
Jackson57279
approved these changes
Aug 16, 2025
|
that looks like a good pr but i dont work for roblox soooo i cant approve it :P |
Anjo2807
approved these changes
Aug 16, 2025
|
Finally, Something not whining about schlep! |
yeah... fucking god just take the #freeschlep thing to twitter and not here |
rafainn
approved these changes
Aug 17, 2025
jbsparrow
suggested changes
Nov 21, 2025
There was a problem hiding this comment.
Please move the "What's new" changes in README.md to a CHANGELOG.md file following https://keepachangelog.com/en/1.1.0/.
|
|
||
| By prioritizing recall over precision, Sentinel serves as a high-recall candidate generator for more thorough investigation. This approach is particularly effective for applications where rare patterns are critical to identify. Rather than treating each message in isolation, Sentinel analyzes patterns across messages to identify concerning behavior. | ||
|
|
||
| ## What’s New: Aggregation options and Explainability |
There was a problem hiding this comment.
This should be in its own file, CHANGELOG.md, and should follow https://keepachangelog.com/en/1.1.0/.
vcai4071
reviewed
Feb 19, 2026
Comment on lines
+262
to
+264
| prevent_exact_match: bool = False, | ||
| encoding_additional_kwargs: Mapping[str, Any] = {}, | ||
| show_progress_bar: bool = False, |
There was a problem hiding this comment.
Could you add back the indentation please?
Comment on lines
+321
to
+325
| # Explainability defaults (always on for transparency) | ||
| explain = True | ||
| include_neighbors = True | ||
| neighbors_limit = 5 | ||
|
|
There was a problem hiding this comment.
Could you add these as parameters with defaults instead please?
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
7 participants
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.
This enhancement adds flexible aggregation strategies and rich explainability to the rare‑class affinity pipeline:
sentinel.score_formulae:RareClassAffinityResult.explanations:aggregation_nameandaggregation_stats(num_texts, num_positive_scores, top_k_per_observation, min_score_to_consider)This keeps the default behavior (skewness + high‑recall) while enabling targeted tuning and greater transparency.
Why this is useful:
All tests are passed: