module: deeptrio

[abhi18av]

PR checklist

Closes #568

• This comment contains a description of changes (with reason).
• If you've fixed a bug or added code that should be tested, add tests!
• If you've added a new tool - have you followed the module conventions in the contribution docs
• If necessary, include test data in your PR.
• Remove all TODO statements.
• Emit the <SOFTWARE>.version.txt file.
• Follow the naming conventions.
• Follow the parameters requirements.
• Follow the input/output options guidelines.
• Add a resource label
• Use BioConda and BioContainers if possible to fulfil software requirements.
• Ensure that the test works with either Docker / Singularity. Conda CI tests can be quite flaky:
  • PROFILE=docker pytest --tag <MODULE> --symlink --keep-workflow-wd
  • PROFILE=singularity pytest --tag <MODULE> --symlink --keep-workflow-wd
  • PROFILE=conda pytest --tag <MODULE> --symlink --keep-workflow-wd

[abhi18av]

Hi @grst , aplogies because of lack of activity here.

I'm still waiting for the usage pattern to emerge on the sarek workflow based on nf-core/sarek#394, and as soon as I have more clarity, I'd finalize this and get this merged.

[grst]

No worries, I was actually just trying to re-trigger the CI which got stuck somehow

…

On Thu., Jul. 22, 2021, 12:03 Abhinav Sharma, ***@***.***> wrote: Hi @grst <https://github.com/grst> , aplogies because of lack of activity here. I'm still waiting for the usage pattern to emerge on the sarek workflow based on nf-core/sarek#394 <nf-core/sarek#394>, and as soon as I have more clarity, I'd finalize this and get this merged. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#573 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABVZRV6HKSNBZ25QPYQXDLDTY7UGRANCNFSM5AKARPFA> .

[abhi18av]

Ah, okay. Got it 👍

[grst]

Getting your branch up-to-date with the latest master did the trick, the CI ran through finally.

[JoseEspinosa]

Hi @abhi18av, thank you for this PR! We are merging as many modules as possible now due to and impending restructuring of the entire repo. This will mean you will need to update the module to reflect these changes before it can be merged in the future. It appears like this module isn't ready to be merged so if applicable, we are converting it to draft and adding the WIP label. If this isn't the case please let us know and we will try to get the module in before the changes. Thanks again!

[matthdsm]

Hi there!

We've noticed there hasn't been much activity here. Are you still planning on working on this?
If not, you can ignore this message and we'll close your PR in about 2 weeks

Cheers
the nf-core maintainers