Skip to content

ToulligQC version 2.9.1#1718

Merged
erinyoung merged 1 commit into
StaPH-B:masterfrom
RaheelSyedAhmed:toulligQC-2.9.1
Jun 26, 2026
Merged

ToulligQC version 2.9.1#1718
erinyoung merged 1 commit into
StaPH-B:masterfrom
RaheelSyedAhmed:toulligQC-2.9.1

Conversation

@RaheelSyedAhmed

Copy link
Copy Markdown
Contributor

Pull Request (PR) checklist:

  • Include a description of what is in this pull request in this message.
  • The dockerfile successfully builds to a test target for the user creating the PR. (i.e. docker build --tag samtools:1.15test --target test docker-builds/build-files/samtools/1.15 )
  • Directory structure as name of the tool in lower case with special characters removed with a subdirectory of the version number in build-files (i.e. docker-builds/build-files/spades/3.12.0/Dockerfile)
    • (optional) All test files are located in same directory as the Dockerfile (i.e. build-files/shigatyper/2.0.1/test.sh)
  • Create a simple container-specific README.md in the same directory as the Dockerfile (i.e. docker-builds/build-files/spades/3.12.0/README.md)
    • If this README is longer than 30 lines, there is an explanation as to why more detail was needed
  • Dockerfile includes the recommended LABELS
  • Main README.md has been updated to include the tool and/or version of the dockerfile(s) in this PR
  • Program_Licenses.md contains the tool(s) used in this PR and has been updated for any missing

PR description:
Updated Dockerfile to use TOULLIGQC_VER=2.9.1. The main README.md has version 2.9.1 added for the versions dropdown entry for ToulligQC. The program specific README.md remains the same; no major syntax changes were seen in the release logs for Toulligqc. The program license remains GNU GPLv3.

… version 2.9.1 added for the versions dropdown entry for ToulligQC. The program specific README.md remains the same; no major syntax changes were seen in the release logs for Toulligqc.
@erinyoung

Copy link
Copy Markdown
Contributor

@staphb-dockerbuilds-diff toulligqc 2.8.4 2.9.1

@github-actions

Copy link
Copy Markdown

Dockerfile Diff: toulligqc

Comparing: 2.8.4 -> 2.9.1

--- build-files/toulligqc/2.8.4/Dockerfile	2026-06-26 18:44:13.571581416 +0000
+++ build-files/toulligqc/2.9.1/Dockerfile	2026-06-26 18:44:13.571697112 +0000
@@ -1,12 +1,12 @@
 FROM python:3.11.11-slim AS app
 
-ARG TOULIGQC_VER="2.8.4"
+ARG TOULLIGQC_VER="2.9.1"
 
 # 'LABEL' instructions tag the image with metadata that might be important to the user
 LABEL base.image="python:3.11.11-slim"
 LABEL dockerfile.version="1"
 LABEL software="ToulligQC"
-LABEL software.version="${TOULIGQC_VER}"
+LABEL software.version="${TOULLIGQC_VER}"
 LABEL description="QC analyses of Oxford Nanopore runs"
 LABEL website="https://github.com/GenomiqueENS/toulligQC"
 LABEL license="https://github.com/GenomiqueENS/toulligQC/blob/master/LICENSE.txt"
@@ -22,9 +22,9 @@
     bzip2 && \
     apt-get autoclean && rm -rf /var/lib/apt/lists/*
 
-RUN wget -q https://github.com/GenomiqueENS/toulligQC/archive/refs/tags/v${TOULIGQC_VER}.tar.gz && \
-    pip install --no-cache-dir v${TOULIGQC_VER}.tar.gz && \
-    rm -rf v${TOULIGQC_VER}.tar.gz
+RUN wget -q https://github.com/GenomiqueENS/toulligQC/archive/refs/tags/v${TOULLIGQC_VER}.tar.gz && \
+    pip install --no-cache-dir v${TOULLIGQC_VER}.tar.gz && \
+    rm -rf v${TOULLIGQC_VER}.tar.gz
 
 # 'ENV' instructions set environment variables that persist from the build into the resulting image
 # Use for e.g. $PATH and locale settings for compatibility with Singularity
@@ -46,7 +46,7 @@
 # A second FROM insruction creates a new stage
 FROM app AS test
 
-ARG TOULIGQC_VER
+ARG TOULLIGQC_VER
 
 # set working directory so that all test inputs & outputs are kept in /test
 WORKDIR /test

@erinyoung

Copy link
Copy Markdown
Contributor

Tests worked

#11 [test 3/3] RUN wget -q http://outils.genomique.biologie.ens.fr/leburon/downloads/toulligqc-example/toulligqc_demo_data.tar.bz2 &&     tar -xvjf toulligqc_demo_data.tar.bz2 &&     cd toulligqc_demo_data &&     ./run-toulligqc.sh &&     ls /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/report.html     /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/report.data     /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/*html
#11 55.60 toulligqc_demo_data/
#11 55.60 toulligqc_demo_data/README
#11 55.60 toulligqc_demo_data/sequencing_telemetry.js
#11 55.64 toulligqc_demo_data/barcoding_summary_fail.txt
#11 58.99 toulligqc_demo_data/run-toulligqc.sh
#11 58.99 toulligqc_demo_data/barcoding_summary_pass.txt
#11 83.76 toulligqc_demo_data/run-toulligqc-with-docker.sh
#11 83.76 toulligqc_demo_data/sequencing_summary.txt
#11 115.2 Warning: Barcode 'B0C7' is non-standard custom arrangement.
#11 132.0 Warning: The barcode B0C7 doesn't exist in input data
#11 149.8 ToulligQC version 2.9.1
#11 149.8 * Initialize extractors
#11 149.8 * Start Toulligqc info extractor
#11 149.8 * End of Toulligqc info extractor (done in 0m0.00s)
#11 149.8 * Start Sequencing telemetry extractor
#11 149.8 * End of Sequencing telemetry extractor (done in 0m0.02s)
#11 149.8 * Start Basecaller sequencing summary extractor
#11 149.8   - Load sequencing summary file (67.13 MB used) in 0m15.35s
#11 149.8   - Extract info from sequencing summary file in 0m2.89s
#11 149.8   - Creation of image "Read count histogram" in 0m0.16s
#11 149.8   - Creation of image "Distribution of read lengths" in 0m1.35s
#11 149.8   - Creation of image "Yield plot through time" in 0m0.77s
#11 149.8   - Creation of image "PHRED score distribution" in 0m2.09s
#11 149.8   - Creation of image "PHRED score density distribution" in 0m0.38s
#11 149.8   - Creation of image "Channel occupancy of the flowcell" in 0m0.20s
#11 149.8   - Creation of image "Correlation between read length and PHRED score" in 0m0.97s
#11 149.8   - Creation of image "Read length over time" in 0m1.73s
#11 149.8   - Creation of image "PHRED score over time" in 0m2.01s
#11 149.8   - Creation of image "Translocation speed" in 0m2.08s
#11 149.8   - Creation of image "Pass barcoded reads distribution" in 0m0.05s
#11 149.8   - Creation of image "Fail barcoded reads distribution" in 0m0.04s
#11 149.8   - Creation of image "Read size distribution for barcodes" in 0m1.64s
#11 149.8   - Creation of image "Read length distribution for barcodes" in 0m1.32s
#11 149.8   - Creation of image "PHRED score distribution for barcodes" in 0m1.56s
#11 149.8 * End of Basecaller sequencing summary extractor (done in 0m34.58s)
#11 149.8 * Write HTML report
#11 149.8 * Write statistics files
#11 149.8 * End of the QC extractor (done in 0m34.64s)
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/Channel_occupancy_of_the_flowcell.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/Correlation_between_read_length_and_PHRED_score.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/Distribution_of_read_lengths.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/Fail_barcoded_reads_distribution.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/PHRED_score_density_distribution.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/PHRED_score_distribution.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/PHRED_score_distribution_for_barcodes.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/PHRED_score_over_time.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/Pass_barcoded_reads_distribution.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/Read_count_histogram.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/Read_length_distribution_for_barcodes.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/Read_length_over_time.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/Read_size_distribution_for_barcodes.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/Translocation_speed.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/images/Yield_plot_through_time.html
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/report.data
#11 150.0 /test/toulligqc_demo_data/output/ToulligQC_Demo_Data/report.html

@erinyoung erinyoung left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have no changes to recommend. Let's get this merged and deployed!

@@ -0,0 +1,65 @@
FROM python:3.11.11-slim AS app

ARG TOULLIGQC_VER="2.9.1"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The "TOULLIGQC_VER" ARG now has two L's. I think it looks better.

@erinyoung erinyoung merged commit 6892891 into StaPH-B:master Jun 26, 2026
2 checks passed
@erinyoung

Copy link
Copy Markdown
Contributor

Thank you for putting this together! You can check the status of the deployment here : https://github.com/StaPH-B/docker-builds/actions/runs/28258993217

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