Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Assertion failure in -fastx_uniques with -uc option #13

Open
PaulGreenfieldOz opened this issue Oct 21, 2024 · 5 comments
Open

Assertion failure in -fastx_uniques with -uc option #13

PaulGreenfieldOz opened this issue Oct 21, 2024 · 5 comments

Comments

@PaulGreenfieldOz
Copy link

Assertion failures when running -fastx_uniques with -uc option. Most useful diagnostic is progress.cpp(496) assert failed: g_State == PS_Idle. Happens on both Windows and Ubuntu builds (20 & 22). fastx_uniques OK if .uc not specified (although sometimes slow, like when -uc specified).

Sample data file:
t_merged_t_dm_CC11A-CC_S7_L001_R1_001.zip

----------------- usearch12 with .uc on Windows ---------------------------------

usearch12 -fastx_uniques t_merged_t_dm_CC11A-CC_S7_L001_R1_001.fa -fastaout t_merged_t_dm_CC11A-CC_S7_L001_R1_001_derep11.fa -uc t_merged_t_dm_CC11A-CC_S7_L001_R1_001_derep11.uc -sizeout

usearch v12.0, 205Gb RAM, 48 cores
(C) Copyright 2013-24 Robert C. Edgar.
https://drive5.com/usearch

fastx_uniques t_merged_t_dm_CC11A-CC_S7_L001_R1_001.fa
00:01 77Mb ...... Loading seqs.
00:01 44Mb ...... CPU has 48 cores, defaulting to 10 threads
00:01 57Mb ...... Unique seqs.
00:01 58Mb ...... 65692 seqs, 18474 uniques, 12583 singletons (68.1%)
00:01 58Mb ...... Min size 1, median 1, max 3721, avg 3.56
00:27 49Mb .. ... Writing 18.5k uniques

A:\Active\USearch12\usearch12-master\src\x64\Debug\usearch12.exe -fastx_uniques t_merged_t_dm_CC11A-CC_S7_L001_R1_001.fa -fastaout t_merged_t_dm_CC11A-CC_S7_L001_R1_001_derep.fa -uc t_merged_t_dm_CC11A-CC_S7_L001_R1_001_derep.uc -sizeout
Elapsed time 00:27
Max memory 77Mb

*** ERROR *** myutils.cpp(252)
A:\Active\USearch12\usearch12-master\src\progress.cpp(496) assert failed: g_State == PS_Idle

@rcedgar
Copy link
Owner

rcedgar commented Oct 21, 2024

Thanks for the report.
I think the bug may be harmless because all output is successfully written?
Using -quiet option may avoid the bug.

@PaulGreenfieldOz
Copy link
Author

Sorry, but the .uc file is created but empty. The fasta file is correct.

regards,
Paul

@PaulGreenfieldOz
Copy link
Author

And using -quiet had no effect - still get assertion failure.

@rcedgar
Copy link
Owner

rcedgar commented Oct 23, 2024

Ok. Suggest falling back to the binary for v11 https://github.com/rcedgar/usearch_old_binaries/

@PaulGreenfieldOz
Copy link
Author

usearch11 works fine - but that doesn't help with migrating a pipeline to usearch12. For now, I'll just write my own derep code and move on.

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

No branches or pull requests

2 participants