eva-sub-cli 0.4.14__tar.gz → 0.5.1__tar.gz

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/.github/workflows/tests.yml

@@ -4,7 +4,7 @@ on:
   push:
     branches: [ main ]
   pull_request:
-    branches: [ main ]
+    branches: [ main, add-validation-tasks ]
 
 jobs:
   build:

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: eva_sub_cli
-Version: 0.4.14
+Version: 0.5.1
 Summary: EBI EVA - validation and submission command line tool
 Classifier: Development Status :: 5 - Production/Stable
 Classifier: Environment :: Console

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/docker/Dockerfile

@@ -1,7 +1,7 @@
 FROM python:3.10
 
 ENV vcf_validator_version=0.10.2
-ENV NXF_VER=22.10.6
+ENV NXF_VER=23.10.0
 
 WORKDIR /opt
 

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/docs/how_to_submit.md

@@ -65,6 +65,21 @@ If you are working with large VCF files and find that validation takes a very lo
 argument `--shallow` to the command, which will validate only the first 10,000 lines in each VCF. Note that running
 shallow validation will **not** be sufficient for actual submission but will allow you to identify most validation concerns without long run time.
 
+### Validation tasks
+To run only a subset of the validation steps, you can use the `--validation_tasks` argument.
+This can be useful if you want to avoid re-running long-running validations that have already passed.
+Note that all validation tasks must pass in order to submit. The report will aggregate the results of previous runs and new ones.
+The possible tasks are:
+* `vcf_check` - includes syntax validation and other checks on VCF files
+* `assembly_check` - includes all checks involving the FASTA file
+* `metadata_check` - includes syntactic and semantic checks on metadata
+* `sample_check` - includes sample coherence checks between VCF files and metadata
+
+For example, to run only `vcf_check` and `sample_check`:
+```shell
+eva-sub-cli.py --metadata_xlsx <metadata file> --submission_dir <submission directory> --validation_tasks vcf_check sample_check
+```
+
 ### VCF files and reference FASTA
 These can be provided either in the metadata file directly, or on the command line using the `--vcf_files` and
 `--reference_fasta` options. Note that if you are using more than one reference FASTA, you **cannot** use the command 

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/docs/input_file_overview.md

@@ -34,12 +34,12 @@ Body: The body of the VCF file contains the actual variant data, with each row r
 # Metadata Spreadsheet
 
 The spreadsheet provides comprehensive contextual information about the dataset, ensuring that each submission is accompanied by detailed descriptions that facilitate proper understanding and use of the data. Key elements included in the metadata spreadsheet are analysis and project information, sample information, sequencing methodologies, experimental details.
-The spreadsheet is composed of editable tabs that will receive the metadata and non-editable helper tabs that provide detailed description of each column.
-
-| WORKSHEET         | EXPLANATION                                                                                                                                                                                                                                                                                                                                                       |
-|-------------------|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
-| Submitter Details | This sheet captures the details of the submitter                                                                                                                                                                                                                                                                                                                  |
-| Project           | The objective of this sheet is to gather general information about the Project including submitter, submitting centre, collaborators, project title, description and publications.                                                                                                                                                                                |
-| Sample            | Projects consist of analyses that are run on samples. We accept sample information in the form of BioSample, ENA or EGA accession(s). We also accept BioSamples sampleset accessions. If your samples are not yet accessioned, and are therefore novel, please use the "Novel sample(s)" sections of the Sample(s) worksheet to have them registered at BioSample |
-| Analysis          | For EVA, each analysis is one vcf file, plus an unlimited number of ancillary files. This sheet allows EVA to link vcf files to a project and to other EVA analyses. Additionally, this worksheet contains experimental meta-data detailing the methodology of each analysis. Important to note; one project can have multiple associated analyses                |
-| Files             | Filenames and associated checking data associated with this EVA submission should be entered into this worksheet. Each file should be linked to exactly one analysis.                                                                                                                                                                                             |
+The spreadsheet is organized into editable tabs, designed for metadata entry, and non-editable helper tabs, which offer detailed explanations and guidance for each column. Users are required to complete all relevant sections within the editable tabs. Mandatory fields in each section are indicated in bold to highlight essential information that must be provided for a valid submission. However, users are strongly encouraged to provide as much additional information as possible to enhance the completeness and usefulness of the metadata.
+
+| WORKSHEET         | EXPLANATION                                                                                                                                                                                                                                                                                                                                                                                                        |
+|-------------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
+| Submitter Details | This sheet captures the details of the submitter                                                                                                                                                                                                                                                                                                                                                                   |
+| Project           | The objective of this sheet is to gather general information about the Project. If you are submitting to an existing project, you can skip the other details and just provide the project accession and analyses will be linked to that project. In case of a new project, please provide the relevant details including submitter, submitting centre, collaborators, project title, description and publications. |
+| Sample            | Projects consist of analyses that are run on samples. We accept sample information in the form of BioSample, ENA or EGA accession(s). We also accept BioSamples sampleset accessions. If your samples are not yet accessioned, and are therefore novel, please use the "Novel sample(s)" sections of the Sample(s) worksheet to have them registered at BioSample                                                  |
+| Analysis          | For EVA, each analysis is one vcf file, plus an unlimited number of ancillary files. This sheet allows EVA to link vcf files to a project and to other EVA analyses. Additionally, this worksheet contains experimental meta-data detailing the methodology of each analysis. Important to note; one project can have multiple associated analyses                                                                 |
+| Files             | Filenames and associated checking data associated with this EVA submission should be entered into this worksheet. Each file should be linked to exactly one analysis.                                                                                                                                                                                                                                              |

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/docs/validation_overview.md

@@ -17,7 +17,7 @@ Key points to note before validating your metadata spreadsheet with the eva-sub-
 
 - Please do not change the existing structure of the spreadsheet
 - Ensure all mandatory columns (marked in bold) are filled.
-- Pre-registered samples must be released and not kept in private status
+- Pre-registered project and samples must be released and not kept in private status
 - Sample names in the spreadsheet must match those in the VCF file.
 - Analysis aliases must match across the sheets (Analysis, Sample, and File sheets).
 

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/eva_sub_cli/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '0.4.14'
-__version_tuple__ = version_tuple = (0, 4, 14)
+__version__ = version = '0.5.1'
+__version_tuple__ = version_tuple = (0, 5, 1)
 
-__commit_id__ = commit_id = 'g1808b05b1'
+__commit_id__ = commit_id = 'g8a5f488cb'

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/eva_sub_cli/etc/eva_schema.json

@@ -64,66 +64,81 @@
       }
     },
     "project": {
-      "type": "object",
-      "description": "General information about the Project",
-      "required": [
-        "title",
-        "description",
-        "taxId",
-        "centre"
-      ],
-      "properties": {
-        "title": {
-          "type": "string",
-          "description": "Title of the project",
-          "maxLength": 500
-        },
-        "description": {
-          "type": "string",
-          "description": "Description of the project",
-          "maxLength": 5000
-        },
-        "centre": {
-          "type": "string",
-          "description": "Centre, institute or company submitting the project"
-        },
-        "taxId": {
-          "type": "number",
-          "description": "Taxonomy ID of the species associated with the project (found here: http://www.ncbi.nlm.nih.gov/taxonomy)"
-        },
-        "publications": {
-          "type": "array",
-          "items": true
-        },
-        "parentProject": {
-          "$ref": "#/definitions/projectAccession"
-        },
-        "childProjects": {
-          "type": "array",
-          "items": {
-            "$ref": "#/definitions/projectAccession"
-          }
-        },
-        "peerProjects": {
-          "type": "array",
-          "items": {
-            "$ref": "#/definitions/projectAccession"
+      "oneOf": [
+        {
+          "type": "object",
+          "required": [
+            "projectAccession"
+          ],
+          "properties": {
+            "projectAccession": {
+              "$ref": "#/definitions/projectAccession"
+            }
           }
         },
-        "links": {
-          "type": "array",
-          "items": {
-            "type": "string"
+        {
+          "type": "object",
+          "description": "General information about the Project",
+          "required": [
+            "title",
+            "description",
+            "taxId",
+            "centre"
+          ],
+          "properties": {
+            "title": {
+              "type": "string",
+              "description": "Title of the project",
+              "maxLength": 500
+            },
+            "description": {
+              "type": "string",
+              "description": "Description of the project",
+              "maxLength": 5000
+            },
+            "centre": {
+              "type": "string",
+              "description": "Centre, institute or company submitting the project"
+            },
+            "taxId": {
+              "type": "number",
+              "description": "Taxonomy ID of the species associated with the project (found here: http://www.ncbi.nlm.nih.gov/taxonomy)"
+            },
+            "publications": {
+              "type": "array",
+              "items": true
+            },
+            "parentProject": {
+              "$ref": "#/definitions/projectAccession"
+            },
+            "childProjects": {
+              "type": "array",
+              "items": {
+                "$ref": "#/definitions/projectAccession"
+              }
+            },
+            "peerProjects": {
+              "type": "array",
+              "items": {
+                "$ref": "#/definitions/projectAccession"
+              }
+            },
+            "links": {
+              "type": "array",
+              "items": {
+                "type": "string"
+              }
+            },
+            "holdDate": {
+              "type": "string",
+              "format": "date",
+              "examples": [
+                "2018-11-13"
+              ]
+            }
           }
-        },
-        "holdDate": {
-          "type": "string",
-          "format": "date",
-          "examples": [
-            "2018-11-13"
-          ]
         }
-      }
+      ]
     },
     "analysis": {
       "type": "array",

eva_sub_cli-0.5.1/eva_sub_cli/etc/spreadsheet2json_conf.yaml

@@ -0,0 +1,139 @@
+#Configuration for defining which fields in which worksheets to be parsed in an Excel file
+#
+#At the top level, "worksheets" defines a list of worksheets (titles) to be parsed.
+#Then for each worksheet, you define which fields to be parsed from this worksheet.
+#These fields are either "required", "optional" and/or header_row. "required" fields must be found
+#in the worksheet so that this worksheet is valid. Otherwise this worksheet is not
+#valid. The field names must match those values in the header row in the worksheet.
+#
+#This is a YAML file
+
+# List of worksheets to be parsed.
+worksheets:
+  Submitter Details: submitterDetails
+  Project: project
+  Analysis: analysis
+  Sample: sample
+  Files: files
+
+Submitter Details:
+  required:
+    Last Name: lastName
+    First Name: firstName
+    Email Address: email
+    Laboratory: laboratory
+    Center: centre
+  optional:
+    Address: address
+
+# List of field for Projects
+Project:
+  header_row: 3
+  optional:
+    Project Accession: projectAccession
+    Project Title: title
+    Description: description
+    Center: centre
+    Taxonomy ID: taxId
+    Publication(s): publications
+    Parent Project: parentProject
+    Child Project(s): childProjects
+    Peer Project(s): peerProjects
+    Link(s): links
+    Hold Date: holdDate
+    Collaborator(s): collaborators
+    Strain: strain
+    Breed: breed
+    Broker: broker
+  cast:
+    Publication(s): list
+    Child Project(s): list
+    Peer Project(s): list
+    Link(s): list
+    Hold Date: date
+    Taxonomy ID: integer
+
+Analysis:
+  required:
+    Analysis Title: analysisTitle
+    Analysis Alias: analysisAlias
+    Description: description
+    Experiment Type: experimentType
+    Reference: referenceGenome
+  optional:
+    Reference Fasta Path: referenceFasta
+    Platform: platform
+    Software: software
+    Pipeline Description: pipelineDescriptions
+    Imputation: imputation
+    Phasing: phasing
+    Centre: centre
+    Date: date
+    Link(s): links
+    Run Accession(s): runAccessions
+  cast:
+    Run Accession(s): list
+    Imputation: boolean
+    Phasing: boolean
+    Date: date
+    Software: list
+
+Sample:
+  header_row: 3
+  required:
+    Analysis Alias: analysisAlias
+    Sample Name in VCF: sampleInVCF
+  # Sample property names taken from ERC000011 where present
+  # See https://www.ebi.ac.uk/biosamples/schema-store/registry/schemas/ERC000011
+  optional:
+    Sample Accession: bioSampleAccession
+    BioSample Name: bioSampleName
+    Title: title
+    Description: description
+    Unique Name: uniqueName
+    Prefix: prefix
+    Subject: subject
+    Derived From: derivedFrom
+    Taxonomy ID: taxId
+    Scientific Name: scientific name
+    Common Name: commonName
+    mating_type: mating_type
+    sex: sex
+    population: population
+    cell_type: cell_type
+    dev_stage: dev_stage
+    germline: germline
+    tissue_lib: tissue_lib
+    tissue_type: tissue_type
+    bio_material: bio_material
+    culture_collection: culture_collection
+    specimen_voucher: specimen_voucher
+    collected_by: collected_by
+    collection_date: collection date
+    geographic location (country and/or sea): geographic location (country and/or sea)
+    geographic location (region and locality): geographic location (region and locality)
+    host: host
+    identified_by: identified_by
+    isolation_source: isolation_source
+    lat_lon: lat_lon
+    lab_host: lab_host
+    environmental_sample: environmental_sample
+    cultivar: cultivar
+    ecotype: ecotype
+    isolate: isolate
+    strain: strain
+    sub_species: sub_species
+    variety: variety
+    sub_strain: sub_strain
+    cell_line: cell_line
+    serotype: serotype
+    serovar: serovar
+  cast:
+    collection_date: date
+    Taxonomy ID: integer
+
+
+Files:
+  required:
+    Analysis Alias: analysisAlias
+    File Name: fileName

eva_sub_cli-0.4.14/eva_sub_cli/etc/spreadsheet2json_conf.yaml → eva_sub_cli-0.5.1/eva_sub_cli/etc/spreadsheet2json_conf_V2.yaml

@@ -50,6 +50,8 @@ Project:
     Peer Project(s): list
     Link(s): list
     Hold Date: date
+    Taxonomy ID: integer
+
 
 Analysis:
   required:
@@ -127,12 +129,12 @@ Sample:
     serotype: serotype
     serovar: serovar
   cast:
-    Sample ID: string
-    BioSample Name: string
     collection_date: date
+    Taxonomy ID: integer
+
 
 
 Files:
   required:
     Analysis Alias: analysisAlias
-    File Name: fileName
+    File Name: fileName

eva_sub_cli-0.5.1/eva_sub_cli/exceptions/submission_upload_exception.py

@@ -0,0 +1,5 @@
+
+class SubmissionUploadException(Exception):
+    def __init__(self, message):
+        self.message = message
+        super().__init__(self.message)

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/eva_sub_cli/executables/cli.py

@@ -1,14 +1,11 @@
 import sys
 
-import eva_sub_cli
-from eva_sub_cli.exceptions.metadata_template_version_exception import MetadataTemplateVersionException, \
-    MetadataTemplateVersionNotFoundException
-from eva_sub_cli.exceptions.submission_not_found_exception import SubmissionNotFoundException
-from eva_sub_cli.exceptions.submission_status_exception import SubmissionStatusException
+from requests import HTTPError
+
+from eva_sub_cli.exceptions.submission_upload_exception import SubmissionUploadException
 
 if not sys.warnoptions:
     import warnings
-
     warnings.simplefilter("ignore")
 
 import logging
@@ -16,9 +13,15 @@ import os
 from argparse import ArgumentParser
 from ebi_eva_common_pyutils.logger import logging_config
 
+import eva_sub_cli
 from eva_sub_cli import orchestrator
-from eva_sub_cli.orchestrator import VALIDATE, SUBMIT, DOCKER, NATIVE
+from eva_sub_cli.exceptions.metadata_template_version_exception import MetadataTemplateVersionException, \
+    MetadataTemplateVersionNotFoundException
+from eva_sub_cli.exceptions.submission_status_exception import SubmissionStatusException
+from eva_sub_cli.exceptions.submission_not_found_exception import SubmissionNotFoundException
 from eva_sub_cli.file_utils import is_submission_dir_writable, DirLockError, DirLock
+from eva_sub_cli.orchestrator import VALIDATE, SUBMIT, DOCKER, NATIVE
+from eva_sub_cli.validators.validator import ALL_VALIDATION_TASKS
 
 
 def validate_command_line_arguments(args, argparser):
@@ -93,6 +96,9 @@ def parse_args(cmd_line_args):
                            help='Select a task to perform (default SUBMIT). VALIDATE will run the validation'
                                 ' regardless of the outcome of previous runs. SUBMIT will run validate only if'
                                 ' the validation was not performed successfully before and then run the submission.')
+    argparser.add_argument('--validation_tasks', nargs='+', choices=ALL_VALIDATION_TASKS, default=ALL_VALIDATION_TASKS,
+                           type=str.lower, help='Select only a subset of the validation tasks to run. Note that all '
+                                                'tasks need to be successful for the validation to pass')
     argparser.add_argument('--executor', choices=[DOCKER, NATIVE], default=NATIVE, type=str.lower,
                            help='Select the execution type for running validation (default native)')
     credential_group = argparser.add_argument_group('Credentials', 'Specify the ENA Webin credentials you want to use '
@@ -125,7 +131,6 @@ def main():
 
     try:
         # lock the submission directory
-
         with DirLock(os.path.join(args.submission_dir)) as lock:
             # Create the log file
             logging_config.add_file_handler(os.path.join(args.submission_dir, 'eva_submission.log'), logging.DEBUG)
@@ -151,4 +156,15 @@ def main():
     except MetadataTemplateVersionNotFoundException as mte:
         print(mte)
         exit_status = 70
+    except SubmissionUploadException as sue:
+        print(sue)
+        exit_status = 71
+    except HTTPError as http_err:
+        print(http_err)
+        if http_err.response is not None and http_err.response.text:
+            print(http_err.response.text)
+        exit_status = 72
+    except Exception as ex:
+        print(ex)
+        exit_status = 73
     return exit_status

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/eva_sub_cli/executables/xlsx2json.py

@@ -103,9 +103,7 @@ class XlsxParser:
     @staticmethod
     def cast_value(value, type_name):
         # Do not cast None values
-        if type_name and value is not None:
-            if type_name == 'string':
-                return str(value)
+        if value is not None:
             if type_name == 'boolean':
                 return str(value).lower() in ['true', '1', "'1'", 't', 'y', 'yes']
             if type_name == 'list':
@@ -114,6 +112,10 @@ class XlsxParser:
                 return [v for v in split_values if v]
             if type_name == 'date':
                 return XlsxParser.serialize(value)
+            if type_name == 'integer':
+                return int(value)
+            # Default any value is cast to string
+            return str(value)
         return value
 
     @staticmethod
@@ -179,15 +181,20 @@ class XlsxParser:
         return rows
 
     def get_project_json_data(self):
+        json_key = self.xlsx_conf[WORKSHEETS_KEY_NAME][PROJECT]
+
         project_data = self.get_rows()
         if len(project_data) > 1:
             logger.warning(f"{PROJECT} worksheet expects a single row of info but more than one found in the file. "
                            f"Only the first row's data will be taken into consideration")
+        elif len(project_data) == 0:
+            logger.error(f"{PROJECT} worksheet expects a single row of info but no info is found in the file.")
+            self.add_error(f"{PROJECT} worksheet expects a single row of info but no info is found in the file.")
+            return {json_key: {}}
 
         first_row = project_data[0]
         first_row.pop('row_num')
 
-        json_key = self.xlsx_conf[WORKSHEETS_KEY_NAME][PROJECT]
         json_value = {self.translate_header(PROJECT, k): v for k, v in first_row.items() if v is not None}
         return {json_key: json_value}
 

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/eva_sub_cli/jinja_templates/html/file_validation.html

@@ -1,6 +1,6 @@
 
 {% macro file_validation_report(validation_results, file_name) -%}
-    {% for check_type, check_per_file in validation_results.items() %}
+    {% for check_type, check_per_file in validation_results.items() if check_type not in ["trim_down", "version"] %}
         {% set result = check_per_file.get(file_name, {}) %}
         {% if check_type == "assembly_check" %}
             {{ assembly_check(result) }}
@@ -20,7 +20,7 @@
         {% set row_class = "report-section fail collapsible" %}
     {% elif error_count > 0 %}
         {% set expand_icon = "▶" %}
-        {% set icon = "❌" %}
+        {% set icon = "❗" %}
         {% set row_class = "report-section warn collapsible" %}
     {% else %}
         {% set icon = "✔" %}

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/eva_sub_cli/jinja_templates/html/report.html

@@ -6,6 +6,13 @@
 {% from 'metadata_validation.html' import metadata_validation_report %}
 {% from 'shallow_validation.html' import optional_shallow_validation_report %}
 
+{% macro validation_not_run_yet_message(text="Process not run yet") %}
+    {% set icon = "⏸" %}   {# pause symbol ⏸ #}
+    {% set row_class = "report-section info" %}
+    {% set expand_icon = "" %}
+    <div class='{{ row_class }}'><span class="expand_icon">{{ expand_icon }}</span> {{ icon }} {{ text }}</div>
+{% endmacro %}
+
 <html lang="EN">
 <head>
     <meta charset="UTF-8">
@@ -63,7 +70,12 @@
         Ensures that required fields are present and values are formatted correctly.
         For requirements, please refer to the <a href="https://www.ebi.ac.uk/eva/?Submit-Data" target=”_blank”>EVA website</a>.
     </div>
-    {{ metadata_validation_report(validation_results) }}
+    {% set run_status = validation_results.get('metadata_check', {}).get('run_status', '') %}
+    {% if run_status %}
+        {{ metadata_validation_report(validation_results) }}
+    {% else %}
+        {{ validation_not_run_yet_message() }}
+    {% endif %}
 </section>
 
 <section>
@@ -72,12 +84,17 @@
         Checks whether each file is compliant with the <a href="http://samtools.github.io/hts-specs/VCFv4.4.pdf" target=”_blank”>VCF specification</a>.
         Also checks whether the variants' reference alleles match against the reference assembly.
     </div>
-    {% for file_name in vcf_files %}
-        {% if file_name != "pass"%}
-        <h3>{{ file_name }}</h3>
-        {{ file_validation_report(validation_results, file_name) }}
-        {% endif %}
-    {% endfor %}
+    {% set run_status = validation_results.get('vcf_check', {}).get('run_status', '') %}
+    {% if run_status %}
+        {% for file_name in vcf_files %}
+            {% if file_name != "pass"%}
+            <h3>{{ file_name }}</h3>
+            {{ file_validation_report(validation_results, file_name) }}
+            {% endif %}
+        {% endfor %}
+    {% else %}
+        {{ validation_not_run_yet_message() }}
+    {% endif %}
 </section>
 
 <section>
@@ -85,7 +102,13 @@
     <div class="description">
         Checks whether information in the metadata is concordant with that contained in the VCF files, in particular sample names.
     </div>
-    {{ sample_name_check_report(validation_results)}}
+    {% set run_status = validation_results.get('sample_check', {}).get('run_status', '') %}
+    {% if run_status %}
+        {{ sample_name_check_report(validation_results)}}
+    {% else %}
+        {{ validation_not_run_yet_message() }}
+
+    {% endif %}
 </section>
 
 <section>
@@ -94,12 +117,17 @@
         Checks that the reference sequences in the FASTA file used to call the variants are accessioned in INSDC.
         Also checks if the reference assembly accession in the metadata matches the one determined from the FASTA file.
     </div>
-    {% for file_name in fasta_files %}
-        {% if file_name != "pass"%}
-        <h3>{{ file_name }}</h3>
-        {{ fasta_check_report(validation_results, file_name) }}
-        {% endif %}
-    {% endfor %}
+    {% set run_status = validation_results.get('fasta_check', {}).get('run_status', '') %}
+    {% if run_status %}
+       {% for file_name in fasta_files %}
+            {% if file_name != "pass"%}
+            <h3>{{ file_name }}</h3>
+            {{ fasta_check_report(validation_results, file_name) }}
+            {% endif %}
+        {% endfor %}
+    {% else %}
+        {{ validation_not_run_yet_message() }}
+    {% endif %}
 </section>
 
 <script>

{eva_sub_cli-0.4.14 → eva_sub_cli-0.5.1}/eva_sub_cli/jinja_templates/html/shallow_validation.html

@@ -2,7 +2,7 @@
 {% macro optional_shallow_validation_report(validation_results) -%}
     {% set results = validation_results.get('shallow_validation', {}) %}
 
-    {% if results.get('required') %}
+    {% if validation_results.get('trim_down') %}
     <section>
         <div class="report-section fail collapsible"> <span class="expand_icon">&#9654;</span>
             &#10060; <b>You requested to run the shallow validation, please run full validation before submitting the data</b>