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

Please update: 2021_PattersonNature #28

Open
2 tasks done
TCLamnidis opened this issue Apr 24, 2024 · 2 comments · Fixed by #31
Open
2 tasks done

Please update: 2021_PattersonNature #28

TCLamnidis opened this issue Apr 24, 2024 · 2 comments · Fixed by #31
Labels
in processing This package is currently being processed by the Minotaur package update

Comments

@TCLamnidis
Copy link
Member

Which package needs changes?

Package name: 2021_PattersonNature

Check for upcoming changes

I confirm there are no changes already suggested for this package that address
my issue in:

What needs changing, and why?

The SSF for the package does not end in a new line, which skipped some of the data when processing.

(optional) What are the changes you'd like?

Add an new line at the SSF EOF, and recreate recipe.

(optional) How did you realise something needs updating?

The eager TSV was missing the last entry.

@TCLamnidis TCLamnidis changed the title 2021_PattersonNature Please update: 2021_PattersonNature Apr 24, 2024
@TCLamnidis TCLamnidis reopened this Apr 24, 2024
@TCLamnidis TCLamnidis added the in processing This package is currently being processed by the Minotaur label Apr 24, 2024
@TCLamnidis
Copy link
Member Author

36 SSF entries are missing a FastQ file for nuclear data. This is likely an issue with the data upload to the ENA, as uploaded BAMs should be converted to FastQs automatically.
May be worthwhile to contact Shop about this.

@TCLamnidis
Copy link
Member Author

Processing update: Issues with automatic fill-in of information to the package janno. Need to manually look through it and understand where the issue arises.

Some individuals have 0-2 SNPs covered on the autosomes. Likely caused by the missing nuclear FastQs mentioned above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in processing This package is currently being processed by the Minotaur package update
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant