Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix multistudies freqency error in ClickHouse #11447

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

fuzhaoyuan
Copy link
Contributor

@fuzhaoyuan fuzhaoyuan commented Mar 12, 2025

Fix #11431 and #11291

Fix profiled count for multistudies by mapping profiled count to its study

In calculation, there are 1 boolean indicator of whether this queried gene has gene panel data, and 3 levels of profiled counts: gene-panel count, (non-gene-panel) WES count, and sample count.

The calculation of displayed profiled count for a single study under all cases are described below:

  • Has gene panel data -> gene-panel count + WES count
  • No gene panel data -> WES count, if WES = 0 then use sample count

For multiple studies:

  • Has gene panel data -> gene-panel count + all WES count
  • No gene panel data, then some studies could have WES count while others don't ->
    • First use sample count for all studies that have no WES count
    • Then add all WES count

Fix alteration count for multistudies by adding study as part of the data object

@fuzhaoyuan fuzhaoyuan self-assigned this Mar 12, 2025
@fuzhaoyuan fuzhaoyuan force-pushed the master-fix-multistudies-frequency-ch branch from 7016ef4 to 9ca5202 Compare March 12, 2025 18:24
@dippindots dippindots added the bug label Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] In combined study scenario, profiled count is not properly reported
2 participants