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

Fail analysis on error in Peddy #2059

Closed
moahaegglund opened this issue Oct 23, 2023 · 4 comments · Fixed by #2066
Closed

Fail analysis on error in Peddy #2059

moahaegglund opened this issue Oct 23, 2023 · 4 comments · Fixed by #2066

Comments

@moahaegglund
Copy link

Is your feature request related to a problem? Please describe.
Today it's routine for prod bioinfo to manually inspect the mip-dna cases in Scout, one part of this is to inspect the sample table to see that there is no exclamation mark under sex or kinship, which means that the sample has failed these checks in Peddy. There is no checks for this in mip-dna today, I think it would be good to include this so that we can remove this manual work from prod bioinfo.

Describe the solution you'd like
Make mip-dna fail on problem with sex and relationships in Peddy.

Additional context
This is a screenshot of the sample table in Scout from the case mentioned in #2057, where sex is correct according to Peddy but reporting failure for the relationships.

Screenshot 2023-10-23 at 11 18 52
@jemten
Copy link
Collaborator

jemten commented Oct 23, 2023

MIP is not under active development and thus this feature will not be implemented in the pipeline. However, the feature you're requesting would be very useful and we will implement it when we bring the new nextflow pipeline into production.

@moahaegglund
Copy link
Author

MIP is not under active development and thus this feature will not be implemented in the pipeline. However, the feature you're requesting would be very useful and we will implement it when we bring the new nextflow pipeline into production.

Sounds great!

@pbiology
Copy link
Contributor

We are now looking to improve the way we do delivery of MIP (and other pipes) cases in Scout, and part of this would be to remove the manual check in Scout by prodbioinfo. For this to work we would need this to be implemented.

The idea of not pushing for MIP but rather nf-core/raredisease is solid, but we should evaluate the timeline hear since it could help out the production quite a bit.

@jemten
Copy link
Collaborator

jemten commented Nov 14, 2023

We could also look into implementing this in CG. Long term we want to have this check in maven or a program using the maven db, but that is probably further away. I can do this check in MIP fairly easy but the qc-checks are moving to CG for the new rd pipeline, pending maven.

@jemten jemten linked a pull request Feb 20, 2024 that will close this issue
6 tasks
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 a pull request may close this issue.

3 participants