Skip to content

Aa ldsc magma#121

Open
LArnoldt wants to merge 15 commits into
mainfrom
aa_ldsc_magma
Open

Aa ldsc magma#121
LArnoldt wants to merge 15 commits into
mainfrom
aa_ldsc_magma

Conversation

@LArnoldt

Copy link
Copy Markdown
Collaborator

No description provided.

LArnoldt and others added 15 commits October 30, 2025 10:31
…dress PR review comments

- Merge make_continuous_annot_from_bimfile/donor_data into make_annot_from_bimfile/donor_data
  via scores= argument; remove the now-redundant continuous-only functions
- Add get_magma_gene_loc to _sldsc_utils: fetches Ensembl gene coordinates in MAGMA headless
  4-column format, cached to ~/cellink_data/magma/
- Add get_dummy_data_paths and get_dummy_onek1k to resources._datasets; replace all hardcoded
  machine paths in tutorials with path-free dummy data calls
- Update magma_analysis_tutorial.ipynb: remove misplaced continuous S-LDSC Part 1 (belongs in
  cell_level_ldsc_analysis_updates.ipynb), add MAGMA binary download cell, replace all hardcoded
  paths (MAGMA_BIN, MAGMA_GENE_LOC, G1000_BFILE) with package downloaders
- Update cell_level_ldsc_analysis_updates.ipynb: replace mygene with pybiomart, add binary vs
  continuous annotation comparison, switch to get_dummy_onek1k
- Update cell_level_ldsc_analysis.ipynb: switch to get_dummy_onek1k, remove hardcoded paths
- Update test_new_functions.py to match merged API

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants