Merged
Conversation
Collaborator
gonzalobenegas
commented
Feb 1, 2026
gonzalobenegas
commented
- Refactors complex traits dataset using latest UKBB finemapping data
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Drop null pip variants at download, inner join methods, and average pips when they agree within 5%. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Average PIPs when both methods agree, use single method when only one available, exclude when they disagree by >5%. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add download_all_finemapping target rule for batch downloads - Add aggregate_traits rule to combine all trait-level results - Compute max PIP across traits and list high-PIP traits - Label positives (PIP > 0.9) and negatives (PIP < 0.01, no nulls) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add build_dataset() function to traitgym.intervals for shared logic - Refactor mendelian_traits_dataset_all and complex_traits_dataset_all rules - Add complex_traits_annotate and complex_traits_full_consequence_counts rules - Add check_ref_alt function to traitgym.variants - Mark slow tests with @pytest.mark.slow (skipped by default) - Fix CHROMS test to expect lexicographic ordering Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Drop ~34 high-PIP variants with very low MAF not present in LD score file during annotation. Add null validation for all matching features in match_features() to catch similar issues early. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Change pull-requests permission back to read to pass workflow validation in the Claude Code Review action. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
gonzalobenegas
force-pushed
the
complex-traits
branch
from
f9a925a to
bba0d3d
Compare
|
No issues found. Checked for bugs and CLAUDE.md compliance. |
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
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.