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

Bug: Empty Outputs in Tumor-Only Analysis with Numbered GEN[1] Fields #54

Open
berntpopp opened this issue Jul 19, 2024 · 2 comments
Open
Assignees
Labels
bug Something isn't working

Comments

@berntpopp
Copy link
Member

Description

Empty outputs are generated in tumor-only analysis when including numbered GEN[1] fields with a higher number than 0. This issue needs to be investigated and resolved to ensure correct output generation.

Steps to Reproduce

  1. Run the script with a tumor-only analysis configuration.
  2. Include numbered GEN[1] fields with a higher number than 0.
  3. Observe that the output is empty.

Expected Behavior

The script should generate the correct output even when numbered GEN[1] fields with a higher number than 0 are included.

Actual Behavior

The output is empty when numbered GEN[1] fields with a higher number than 0 are included in tumor-only analysis.

Proposed Solution

Investigate and fix the issue causing empty outputs in this scenario.

Impact

Fixing this issue will ensure that the script generates the correct output for tumor-only analysis, even when including numbered GEN[1] fields with higher numbers.

Additional Notes

  • Ensure backward compatibility with existing functionality.
  • Update the documentation to reflect the correct handling of numbered GEN[1] fields in tumor-only analysis.
@berntpopp berntpopp added the bug Something isn't working label Jul 19, 2024
@berntpopp berntpopp self-assigned this Jul 19, 2024
@berntpopp
Copy link
Member Author

SnpSift extractFields odes not handle missing indices well and has no setting for this, thus keep wildcards only (maybe document this) and handle splitting posthoc (Excel generation).
We should add some kind of check for indices >0.

@berntpopp
Copy link
Member Author

berntpopp commented Sep 5, 2024

Catch errors from the main command and log them for debugging.

Relates to #52

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant