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

Clean up code #9

Merged
merged 22 commits into from
Oct 10, 2024
Merged

Clean up code #9

merged 22 commits into from
Oct 10, 2024

Conversation

nbtm-sh
Copy link
Collaborator

@nbtm-sh nbtm-sh commented Aug 8, 2024

  • Removing hardcoded paths and adding sensible defaults

nbtm-sh added 22 commits August 8, 2024 16:04
…he script does not attempt to modify the existing database file
@nbtm-sh nbtm-sh merged commit 8ce0598 into unsw-dev Oct 10, 2024
Copy link

github-actions bot commented Oct 10, 2024

nf-core lint overall result: Failed ❌

Posted for pipeline commit 9af55b1

+| ✅ 153 tests passed       |+
!| ❗  17 tests had warnings |!
-| ❌  52 tests failed       |-

❌ Test failures:

  • schema_params - Param bfd_prefix from nextflow config not found in nextflow_schema.json
  • schema_params - Param smallbfd_prefix from nextflow config not found in nextflow_schema.json
  • schema_params - Param mgnify_prefix from nextflow config not found in nextflow_schema.json
  • schema_params - Param pdb70_prefix from nextflow config not found in nextflow_schema.json
  • schema_params - Param pdb_mmcif_prefix from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniclust30_prefix from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniref90_prefix from nextflow config not found in nextflow_schema.json
  • schema_params - Param pdb_seq_prefix from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniprot_prefix from nextflow config not found in nextflow_schema.json
  • schema_params - Param alphafold_params_prefix from nextflow config not found in nextflow_schema.json
  • schema_params - Param mmcif_path from nextflow config not found in nextflow_schema.json
  • schema_params - Param mmcif_obsolete from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniclust30_db from nextflow config not found in nextflow_schema.json
  • schema_params - Param bfd_first_non_consensus_sequences from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniprot_fasta from nextflow config not found in nextflow_schema.json
  • schema_params - Param pdb_seqres_txt from nextflow config not found in nextflow_schema.json
  • schema_params - Param bfd_metaclust_clu_complete_id30_c90_final_seq_sorted_opt from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniref90_fasta from nextflow config not found in nextflow_schema.json
  • schema_params - Param mgy_clusters_fasta from nextflow config not found in nextflow_schema.json
  • schema_params - Param bfd_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param smallbfd_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param mgnify_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param pdb70_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param pdb_mmcif_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniclust30_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniref90_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param pdb_seqres_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniprot_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param alphafold_params_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param mmcif_files_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param mmcif_obsolete_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniclust30_db_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param bfd_first_non_consensus_sequences_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniprot_fasta_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param pdb_seqres_txt_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param bfd_metaclust_clu_complete_id30_c90_final_seq_sorted_opt_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniref90_fasta_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param mgy_clusters_fasta_name from nextflow config not found in nextflow_schema.json
  • schema_params - Param cpuQueue from nextflow config not found in nextflow_schema.json
  • schema_params - Param gpuQueue from nextflow config not found in nextflow_schema.json
  • schema_params - Param bfd_dir_path from nextflow config not found in nextflow_schema.json
  • schema_params - Param small_bfd_dir_path from nextflow config not found in nextflow_schema.json
  • schema_params - Param mgnify_dir_path from nextflow config not found in nextflow_schema.json
  • schema_params - Param pdb70_dir_path from nextflow config not found in nextflow_schema.json
  • schema_params - Param pdb_mmcif_dir_path from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniclust30_dir_path from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniref90_dir_path from nextflow config not found in nextflow_schema.json
  • schema_params - Param pdb_seqres_dir_path from nextflow config not found in nextflow_schema.json
  • schema_params - Param uniprot_dir_path from nextflow config not found in nextflow_schema.json
  • schema_params - Param mgnify_database_path from nextflow config not found in nextflow_schema.json
  • schema_params - Param template_mmcif_dir from nextflow config not found in nextflow_schema.json
  • schema_params - Param obsolete_pdbs_path from nextflow config not found in nextflow_schema.json

❗ Test warnings:

  • files_exist - File not found: conf/igenomes.config
  • files_exist - File not found: lib/WorkflowProteinfold.groovy
  • nextflow_config - Config manifest.version should end in dev: '1.0.0'
  • pipeline_todos - TODO string in README.md: Write a 1-2 sentence summary of what data the pipeline is for and what it does
  • pipeline_todos - TODO string in README.md: Add full-sized test dataset and amend the paragraph below if applicable
  • pipeline_todos - TODO string in README.md: Add citation for pipeline after first release. Uncomment lines below and update Zenodo doi and badge at the top of this file.
  • pipeline_todos - TODO string in README.md: Add bibliography of tools and data used in your pipeline
  • pipeline_todos - TODO string in usage.md: Add documentation about anything specific to running your pipeline. For general topics, please point to (and add to) the main nf-core website.
  • pipeline_todos - TODO string in katana.config: Check the defaults for all processes
  • pipeline_todos - TODO string in katana.config: Customise requirements for specific processes.
  • pipeline_todos - TODO string in base.config: Check the defaults for all processes
  • pipeline_todos - TODO string in base.config: Customise requirements for specific processes.
  • pipeline_todos - TODO string in methods_description_template.yml: #Update the HTML below to your prefered methods description, e.g. add publication citation for this pipeline
  • pipeline_todos - TODO string in check_samplesheet.py: Update the script to check the samplesheet
  • pipeline_todos - TODO string in check_samplesheet.py: Update the check_samplesheet function
  • pipeline_todos - TODO string in check_samplesheet.py: Update the column names for the input samplesheet
  • pipeline_todos - TODO string in WorkflowMain.groovy: Add Zenodo DOI for pipeline after first release

✅ Tests passed:

Run details

  • nf-core/tools version 2.7.2
  • Run at 2024-10-10 01:28:46

@keiran-rowell-unsw keiran-rowell-unsw deleted the clean-up-code branch October 11, 2024 04:29
jscgh pushed a commit that referenced this pull request Nov 11, 2024
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.

1 participant