Skip to content

Improve xchr function(AlphaGenes/AlphaPeel#225)#226

Open
AprilYUZhang wants to merge 11 commits intoAlphaGenes:develfrom
AprilYUZhang:feat/issue-225-improve-xchr-function
Open

Improve xchr function(AlphaGenes/AlphaPeel#225)#226
AprilYUZhang wants to merge 11 commits intoAlphaGenes:develfrom
AprilYUZhang:feat/issue-225-improve-xchr-function

Conversation

@AprilYUZhang
Copy link
Copy Markdown
Contributor

@AprilYUZhang AprilYUZhang commented Mar 3, 2026

Related Issue

Relate to #225

What changed

  • Figure out the issue with the simple example
  • Modify penetrance function

Why this change

See #199 #216 #217

Notes / Risks

For x chr, the default seg probs are changed.

@XingerTang XingerTang marked this pull request as draft March 5, 2026 10:56
@gregorgorjanc
Copy link
Copy Markdown
Member

@AprilYUZhang how do these changes impact the example results - can you share like I did at #199 (comment)?

@XingerTang
Copy link
Copy Markdown
Contributor

Hi @AprilYUZhang, can you provide a tinyhouse reference that this code can run with?

@AprilYUZhang
Copy link
Copy Markdown
Contributor Author

e8a7cd3e12378941e2aef5a05ff161eb49204a9f

…rove-xchr-function

 Merge latest devel branch
@AprilYUZhang
Copy link
Copy Markdown
Contributor Author

I have merged the latest devel branch, and it works.

@AprilYUZhang
Copy link
Copy Markdown
Contributor Author

Have done.

@XingerTang XingerTang marked this pull request as ready for review April 9, 2026 14:34
@XingerTang
Copy link
Copy Markdown
Contributor

All looks good to me! @AprilYUZhang, can you squash some commits, or do you prefer to merge directly?

@gregorgorjanc
Copy link
Copy Markdown
Member

Best to squash commits if possible.

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.

3 participants