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

Add Influenza isolate subtyping #645

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

SaimMomin12
Copy link

Hello IWC Team,

This is my first contribution to IWC. This pull request introduces a new workflow designed for the subtyping of Avian Influenza isolates.

I welcome your suggestions and reviews to improve this contribution.

Thanks!


FOR CONTRIBUTOR:

  • I have read the Adding workflows guidelines
  • License permits unrestricted use (educational + commercial)
  • Please also take note of the reviewer guidelines below to facilitate a smooth review process.

FOR REVIEWERS:

  • .dockstore.yml: file is present and aligned with creator metadata in workflow. ORCID identifiers are strongly encouraged in creator metadata. The .dockstore.yml file is required to run tests
  • Workflow is sufficiently generic to be used with lab data and does not hardcode sample names, reference data and can be run without reading an accompanying tutorial.
  • In workflow: annotation field contains short description of what the workflow does. Should start with This workflow does/runs/performs … xyz … to generate/analyze/etc …
  • In workflow: workflow inputs and outputs have human readable names (spaces are fine, no underscore, dash only where spelling dictates it), no abbreviation unless it is generally understood. Altering input or output labels requires adjusting these labels in the the workflow-tests.yml file as well
  • In workflow: name field should be human readable (spaces are fine, no underscore, dash only where spelling dictates it), no abbreviation unless generally understood
  • Workflow folder: prefer dash (-) over underscore (_), prefer all lowercase. Folder becomes repository in iwc-workflows organization and is included in TRS id
  • Readme explains what workflow does, what are valid inputs and what outputs users can expect. If a tutorial or other resources exist they can be linked. If a similar workflow exists in IWC readme should explain differences with existing workflow and when one might prefer one workflow over another
  • Changelog contains appropriate entries
  • Large files (> 100 KB) are uploaded to zenodo and location urls are used in test file

Copy link

github-actions bot commented Jan 9, 2025

Test Results (powered by Planemo)

Test Summary

Test State Count
Total 1
Passed 0
Error 1
Failure 0
Skipped 0
Errored Tests
  • ❌ influenza-isolate-subtyping.ga_0

    Execution Problem:

    • Unexpected HTTP status code: 400: {"err_msg":"Field required in ('body', 'element_identifiers', 0, 'element_identifiers', 0, 'src')","err_code":400007,"validation_errors":["{\"type\": \"missing\", \"loc\": [\"body\", \"element_identifiers\", 0, \"element_identifiers\", 0, \"src\"], \"msg\": \"Field required\", \"input\": {\"identifier\": \"forward\", \"class\": \"File\", \"lication\": \"https://zenodo.org/records/14620360/files/SRR24839074_1.fastq\", \"filetype\": \"fastqsanger\", \"name\": \"forward\"}}"]}
      

Copy link

github-actions bot commented Jan 9, 2025

Test Results (powered by Planemo)

Test Summary

Test State Count
Total 1
Passed 0
Error 1
Failure 0
Skipped 0
Errored Tests
  • ❌ influenza-isolate-subtyping.ga_0

    Execution Problem:

    • Unexpected HTTP status code: 400: {"err_msg":"Field required in ('body', 'element_identifiers', 0, 'element_identifiers', 0, 'src')","err_code":400007,"validation_errors":["{\"type\": \"missing\", \"loc\": [\"body\", \"element_identifiers\", 0, \"element_identifiers\", 0, \"src\"], \"msg\": \"Field required\", \"input\": {\"identifier\": \"forward\", \"class\": \"File\", \"lication\": \"https://zenodo.org/records/14620360/files/SRR24839074_1.fastq\", \"filetype\": \"fastqsanger\", \"name\": \"forward\"}}"]}
      

1 similar comment
Copy link

github-actions bot commented Jan 9, 2025

Test Results (powered by Planemo)

Test Summary

Test State Count
Total 1
Passed 0
Error 1
Failure 0
Skipped 0
Errored Tests
  • ❌ influenza-isolate-subtyping.ga_0

    Execution Problem:

    • Unexpected HTTP status code: 400: {"err_msg":"Field required in ('body', 'element_identifiers', 0, 'element_identifiers', 0, 'src')","err_code":400007,"validation_errors":["{\"type\": \"missing\", \"loc\": [\"body\", \"element_identifiers\", 0, \"element_identifiers\", 0, \"src\"], \"msg\": \"Field required\", \"input\": {\"identifier\": \"forward\", \"class\": \"File\", \"lication\": \"https://zenodo.org/records/14620360/files/SRR24839074_1.fastq\", \"filetype\": \"fastqsanger\", \"name\": \"forward\"}}"]}
      

Copy link

github-actions bot commented Jan 9, 2025

Test Results (powered by Planemo)

Test Summary

Test State Count
Total 1
Passed 0
Error 1
Failure 0
Skipped 0
Errored Tests
  • ❌ influenza-isolate-subtyping.ga_0

    Execution Problem:

    • Unexpected HTTP status code: 400: {"err_msg":"Field required in ('body', 'element_identifiers', 0, 'element_identifiers', 0, 'src')","err_code":400007,"validation_errors":["{\"type\": \"missing\", \"loc\": [\"body\", \"element_identifiers\", 0, \"element_identifiers\", 0, \"src\"], \"msg\": \"Field required\", \"input\": {\"identifier\": \"forward\", \"class\": \"File\", \"lication\": \"https://zenodo.org/records/14620360/files/SRR24839074_1.fastq\", \"filetype\": \"fastqsanger\", \"name\": \"forward\"}}"]}
      

1 similar comment
Copy link

github-actions bot commented Jan 9, 2025

Test Results (powered by Planemo)

Test Summary

Test State Count
Total 1
Passed 0
Error 1
Failure 0
Skipped 0
Errored Tests
  • ❌ influenza-isolate-subtyping.ga_0

    Execution Problem:

    • Unexpected HTTP status code: 400: {"err_msg":"Field required in ('body', 'element_identifiers', 0, 'element_identifiers', 0, 'src')","err_code":400007,"validation_errors":["{\"type\": \"missing\", \"loc\": [\"body\", \"element_identifiers\", 0, \"element_identifiers\", 0, \"src\"], \"msg\": \"Field required\", \"input\": {\"identifier\": \"forward\", \"class\": \"File\", \"lication\": \"https://zenodo.org/records/14620360/files/SRR24839074_1.fastq\", \"filetype\": \"fastqsanger\", \"name\": \"forward\"}}"]}
      

Copy link

github-actions bot commented Jan 9, 2025

Test Results (powered by Planemo)

Test Summary

Test State Count
Total 1
Passed 0
Error 1
Failure 0
Skipped 0
Errored Tests
  • ❌ influenza-isolate-subtyping.ga_0

    Execution Problem:

    • Failed to upload data, upload state is [error].
      

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