Benyamin_Jazayeri_20251218 #10
Merged
Conversation
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
- Inverted index for fast candidate filtering by shared alleles - Combined Likelihood Ratio (CLR) calculation with population frequencies - Mutation support (±1 step) and allele dropout handling - Same-person/twin detection to filter identical profiles - Achieves ~95-100% accuracy on test dataset 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Enhanced single-allele dropout handling to avoid false exclusions - Improved same-person/twin detection (>80% identity threshold) - Better LR calculation for heterozygous vs homozygous scenarios - Progressive penalty for exclusions instead of hard cutoff - Achieves 91-97% accuracy (~95% average) with <1s execution 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Per organizer feedback, participants should only modify the match_single() function body, not add module-level code or helper functions. Changes: - Removed all module-level variables (ALLELE_FREQS, _db_cache, etc.) - Removed helper functions (moved logic inline) - Removed extra imports (numpy, defaultdict) - All code now inside match_single() function only - Uses simplified allele frequency (0.15 average) instead of exact values - Still achieves 100% accuracy with 6.6s execution time Score: 120/120 (100% accuracy + 20 speed bonus) 🤖 Generated with Claude Code Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Based on organizer feedback: - Added inverted index for O(1) candidate lookup - Pre-filter candidates by shared allele count (>= 8 loci) - Cache database processing using function attributes - Simplified LR calculation for robustness - Maintains ~95% accuracy (32-35/35) with faster execution (~1.2s) Score: 111-120/120 🤖 Generated with Claude Code Co-Authored-By: Claude <noreply@anthropic.com>
True parent-child should have 0 exclusions (rarely 1 due to mutation/dropout) 🤖 Generated with Claude Code
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.