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

basic adaptations to work on FLI cluster as well #3

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

johannesgeibel
Copy link

These changes were necessary to make the pipeline work on the FLI cluster. They included some adaptations to work on Slurm with Singularity containers (exporting of certain file systems and TMPDIR paths, queue naming) and setting file paths. Additionally, there remain some further questions before merging the branch:

  • how to include additional clair 3 models in the best way
  • Porechop_abi caused an error due to not renaming splitted reads correctly --> could be a problem of having methylation tags in the fastq. Still need to check and potentially open an issue for porechop_abi
  • mosdepth curretly writes output to "Log" --> should write to "QC" instead?
  • It would be helpful to add a final rule for multiqc to summarize QC results across samples
  • setting of paths should be at the beginning of the "nextflow.config" file, as needs likely to be changed
  • pipeline contains "meta/metadata_LR.csv", which needed to be renamed to "meta/LR_metadata.csv" --> should come from a parameter set in config instead
  • for the BOV_LRC, certain participants might have their raw data as u.bam format --> should we provide a script to do bam to fastq transvertion, or include a process in pipeline?

@tuannguyen8390 tuannguyen8390 self-assigned this Nov 13, 2023
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