ImagoXV / NanoASV

NanoASV official repo
GNU General Public License v3.0
5 stars 0 forks source link

warnings with nanoasv --dry-run #107

Open frederic-mahe opened 1 day ago

frederic-mahe commented 1 day ago

(running on a fresh ubuntu-server-24-04, with Miniconda3)

/home/fred/NanoASV/workflow/run.sh: line 252: [: missing `]'
Checking reference format
/home/fred/NanoASV/workflow/snakefile:200: SyntaxWarning: invalid escape sequence '\+'
  "tmp_files/08_phylogeny/.phylogeny.done"
/home/fred/NanoASV/workflow/snakefile:222: SyntaxWarning: invalid escape sequence '\+'
  "tmp_files/.export.done"

same thing with nanoasv --mock

ImagoXV commented 11 hours ago

I know, I'm not sure what is triggering this warning. Maybe the fact that those are hidden files ? I can remove the hidden status, it does not matter, it is in tmp files in the end

Edit : Removing hidden status didn't cure

/home/user/NanoASV/workflow/snakefile:223: SyntaxWarning: invalid escape sequence '\+' 
"tmp_files/export.done" 

And there is "+" sign in those lines neither.

The only + written in code is

in snakemake file : grep -o '[^ ]\+$' tmp_files/06_sam_to_bam_format/{wildcards.barcode}_abundance.tsv > {output.ASV_list} But it is currently on lines 115 and 118. They have no link with the error and should be safe.

If you have any idea, I'm taking

ImagoXV commented 11 hours ago

For line 252 related error, it comes from the way I wrote echo functions. This is fixed in d70701a3086b798d46a97e5b5ed399661acc9edb and a37c736d6d9f67589a970db38a32999ef0bb8d30