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

Mf snakemake vinayakvsv #25

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

vinayakvsv
Copy link

Modifications made to Snakemake workflow

  1. mappability, repeats, segdup tracks, reference FASTA, and desired coverage for model now added as parameters on config.yaml for easier modification.
  2. fixed an error in how DEL events were being extracted prior to analysis
  3. import numpy as np missing from Phase.py

vinayakvsv added 5 commits November 27, 2021 18:50
…to config file. Moved coverage variable to config file. Need to update with a script that can download the necessary tracks of repeats, segdups, mappability, etc.
@vinayakvsv
Copy link
Author

One other error that I have been running into -- especially during parallel runs of the Snakemake, the ldpaths file in the local conda folder for MF keeps getting deleted during the Phase stage, so the Prediction_SNV rule often throws an error. I am not sure why, but we will need to find a fix for this (likely needs to come from the Anaconda people themselves).

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.

1 participant