search

LooseLab / readfish

CLI tool for flexible and fast adaptive sampling on ONT sequencers
https://looselab.github.io/readfish/
GNU General Public License v3.0
164 stars 31 forks source link

readfish stats crashes without indicating why if fed promethION data without the --prom flag #285

Closed mattloose closed 9 months ago

mattloose commented 9 months ago

readfish stats should warn the user that a channel number has been encountered that is higher than expected and suggest using the --prom flag.

Adoni5 commented 9 months ago

Can you include the error please

mattloose commented 9 months ago 
2023-10-11 11:31:23,843 readfish.stats Loaded TOML config without error                                                                         
Aligning FASTQ, please wait  |                  ▶▶▶▶▶▶▶▶▶▶▶▶▶▶        | ⠠⢀Y (••) Y⡀⡀⢀⠄⡀ 0 in 2s (0.0/s) Internal error returning data, the recei
ver iterator has finished. sending on a disconnected channel 750                                                                                
Aligning FASTQ, please wait  |████████████████████████████████████████| 1 in 2.5s (0.43/s)                                                      
2023-10-11 11:31:46,406 readfish.stats Fastq data couldn't be summarised, see below for details:                                                
2023-10-11 11:31:46,409 readfish.stats 2367
Adoni5 commented 9 months ago

Fixed in https://github.com/Adoni5/ReadfishSummarise/commit/569cfa3acc568132756ff35ca5489b1730f1c3d8