--- a/bbduk.xml	Sat May 18 19:13:38 2024 +0000
+++ b/bbduk.xml	Tue Aug 27 10:15:37 2024 +0000
@@ -154,11 +154,11 @@
             </param>
             <when value="no_reference"/>
             <when value="files">
-                <param name="reference" type="data" format="fasta,fasta.gz" multiple="true" optional="false" label="Select one or more fasta file"/>
+                <param name="reference" type="data" format="fasta,fasta.gz" optional="false" label="Select one or more fasta file" multiple="true"/>
                 <expand macro="ktrim_cond"/>
             </when>
             <when value="keywords">
-                <param name="reference"  type="select" multiple="true" optional="false" label="Select one or more keywords">
+                <param name="reference" type="select" optional="false" label="Select one or more keywords" multiple="true">
                     <option value="adapters">adapters</option>
                     <option value="artifacts">artifacts</option>
                     <option value="phix">phix</option>
@@ -171,34 +171,34 @@
             </when>
         </conditional>
         <section name="advanced_options" title="Advanced options" expanded="false">
-            <param argument="k" type="integer" value="27" min="1" label="Kmer length used for finding contaminants"/>
+            <param argument="k" type="integer" min="1" value="27" label="Kmer length used for finding contaminants"/>
             <param argument="rcomp" type="boolean" truevalue="t" falsevalue="f" checked="true" label="Look for reverse-complements of kmers in addition to forward kmers?"/>
             <param argument="maskmiddle" type="boolean" truevalue="t" falsevalue="f" checked="true" label="Treat the middle base of a kmer as a wildcard to increase sensitivity in the presence of errors?"/>
-            <param argument="minkmerhits" type="integer" value="1" min="1" label="Reads need at least this many matching kmers to be considered as matching the reference"/>
-            <param argument="minkmerfraction" type="float" value="0" min="0" label="A read needs at least this fraction of its total kmers to hit a ref in order to be considered a match"/>
-            <param argument="mincovfraction" type="float" value="0" min="0" label="A read needs at least this fraction of its total bases to be covered by ref kmers to be considered a match"/>
-            <param argument="hammingdistance" type="integer" value="0" min="0" label="Maximum Hamming distance for ref kmers (subs only)"/>
-            <param argument="qhdist" type="integer" value="0" min="0" label="Hamming distance for query kmers"/>
-            <param argument="editdistance" type="integer" value="0" min="0" label="Maximum edit distance from ref kmers (subs and indels)"/>
+            <param argument="minkmerhits" type="integer" min="1" value="1" label="Reads need at least this many matching kmers to be considered as matching the reference"/>
+            <param argument="minkmerfraction" type="float" min="0" value="0" label="A read needs at least this fraction of its total kmers to hit a ref in order to be considered a match"/>
+            <param argument="mincovfraction" type="float" min="0" value="0" label="A read needs at least this fraction of its total bases to be covered by ref kmers to be considered a match"/>
+            <param argument="hammingdistance" type="integer" min="0" value="0" label="Maximum Hamming distance for ref kmers (subs only)"/>
+            <param argument="qhdist" type="integer" min="0" value="0" label="Hamming distance for query kmers"/>
+            <param argument="editdistance" type="integer" min="0" value="0" label="Maximum edit distance from ref kmers (subs and indels)"/>
             <param argument="forbidn" type="boolean" truevalue="t" falsevalue="f" checked="false" label="Do not match kmers comntaining N?"/>
             <param argument="trimfailures" type="boolean" truevalue="t" falsevalue="f" checked="false" label="Trim failed reads to 1bp instead of discarding them?"/>
             <param argument="findbestmatch" type="boolean" truevalue="t" falsevalue="f" checked="false" label="Associate read with sequence sharing most kmers if multiple matches?"/>
             <param argument="skipr1" type="boolean" truevalue="t" falsevalue="f" checked="false" label="Don't do kmer-based operations on read 1?"/>
             <param argument="skipr2" type="boolean" truevalue="t" falsevalue="f" checked="false" label="Don't do kmer-based operations on read 2?"/>
         </section>
-        <param name="outputs_select"  type="select" multiple="true" optional="false" label="Specify outputs">
+        <param name="outputs_select" type="select" optional="false" label="Specify outputs" multiple="true">
             <option value="outu">Unmatched</option>
             <option value="outm">Matched</option>
             <option value="outs">Single</option>
         </param>
         <conditional name="output_stats_cond">
-            <param name="output_stats"  type="select" label="Output statistics?">
+            <param name="output_stats" type="select" label="Output statistics?">
                 <option value="no" selected="true">No</option>
                 <option value="yes">Yes</option>
             </param>
             <when value="no"/>
             <when value="yes">
-                <param name="output_stats_select"  type="select" multiple="true" optional="false" label="Specify statistics outputs">
+                <param name="output_stats_select" type="select" optional="false" label="Specify statistics outputs" multiple="true">
                     <option value="stats">Statistics about which contamininants were detected</option>
                     <option value="ref">Statistics on a per-reference-file basis</option>
                     <option value="rpkm">RPKM for each reference sequence (for RNA-seq)</option>
@@ -207,13 +207,13 @@
             </when>
         </conditional>
         <conditional name="output_hists_cond">
-            <param name="output_hists"  type="select" label="Output histograms?">
+            <param name="output_hists" type="select" label="Output histograms?">
                 <option value="no" selected="true">No</option>
                 <option value="yes">Yes</option>
             </param>
             <when value="no"/>
             <when value="yes">
-                <param name="output_hists_select"  type="select" multiple="true" optional="false" label="Specify statistics outputs">
+                <param name="output_hists_select" type="select" optional="false" label="Specify statistics outputs" multiple="true">
                     <option value="bhist">Base composition histogram by position</option>
                     <option value="quhist">Quality histogram by position</option>
                     <option value="quchist">Count of bases with each quality value</option>
@@ -229,7 +229,7 @@
     </inputs>
     <outputs>
         <data name="outputu" format="fastqsanger" label="${tool.name} on ${on_string} (Forward Unmatched)">
-            <filter>str(outputs_select).find('outu') >= 0</filter>
+            <filter>str(outputs_select).find('outu') &gt;= 0</filter>
             <filter>'outu' in outputs_select</filter>
         </data>
         <data name="outputu2" format="fastqsanger" label="${tool.name} on ${on_string} (Reverse Unmatched)">
@@ -350,4 +350,3 @@
     </help>
     <expand macro="citations"/>
 </tool>
-