Skip to content

Some changes to review#6

Closed
gavin-peng wants to merge 11 commits intoLincolnSteinLab:mainfrom
oicr-gsi:main
Closed

Some changes to review#6
gavin-peng wants to merge 11 commits intoLincolnSteinLab:mainfrom
oicr-gsi:main

Conversation

@gavin-peng
Copy link
Copy Markdown
Contributor

@gavin-peng gavin-peng commented Oct 1, 2025

1, Added Pipefile for the packages, some version upgraded
2, Change in utils.py, this is trying to fix the issue when saw error:

Traceback (most recent call last):
  File "/.mounts/labs/gsi/modulator/sw/Ubuntu20.04/deep-tumour-3.0.3/src/DeepTumour.py", line 222, in <module>
    DeepTumour()
  File "/.mounts/labs/gsi/modulator/sw/Ubuntu20.04/deep-tumour-3.0.3/.venv/lib/python3.10/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/.mounts/labs/gsi/modulator/sw/Ubuntu20.04/deep-tumour-3.0.3/.venv/lib/python3.10/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/.mounts/labs/gsi/modulator/sw/Ubuntu20.04/deep-tumour-3.0.3/.venv/lib/python3.10/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/.mounts/labs/gsi/modulator/sw/Ubuntu20.04/deep-tumour-3.0.3/.venv/lib/python3.10/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/.mounts/labs/gsi/modulator/sw/Ubuntu20.04/deep-tumour-3.0.3/src/DeepTumour.py", line 173, in DeepTumour
    input = vcf2input(vcfFile, refGenome, hg38)
  File "/.mounts/labs/gsi/modulator/sw/Ubuntu20.04/deep-tumour-3.0.3/src/utils.py", line 265, in vcf2input
    mutations:pd.DataFrame = df2mut(df, sample_name, fasta)
  File "/.mounts/labs/gsi/modulator/sw/Ubuntu20.04/deep-tumour-3.0.3/src/utils.py", line 203, in df2mut
    if (ref != ref_ctx[1]):
IndexError: string index out of range

The error came after we added step for filtering vcf calls, using bed file generated from filtering maf, and related to coordinates conversion:
https://github.com/oicr-gsi/deepTumourWorkflow/blob/gavin_updates/deepTumour.wdl#L121

@GFJHogue
Copy link
Copy Markdown
Collaborator

GFJHogue commented Oct 2, 2025

Just making a couple of tweaks to this in #7

@GFJHogue GFJHogue closed this Oct 2, 2025
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