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

segfault with pore-c-py annotate #8

Open
ahcm opened this issue Jan 28, 2024 · 5 comments
Open

segfault with pore-c-py annotate #8

ahcm opened this issue Jan 28, 2024 · 5 comments

Comments

@ahcm
Copy link

ahcm commented Jan 28, 2024

[04:46:22 - AnntateAln] Found 427679179 monomers in 28247142 concatemers.
Segmentation fault (core dumped)
srun: error: HOSTNAME: task 0: Exited with exit code 139
@ahcm
Copy link
Author

ahcm commented Jan 30, 2024

Running again, this time with:

export NUMEXPR_MAX_THREADS=64
pore-c-py annotate --threads 64

Results the same output (size) and also segfaults.
Another dataset also segfaults.

@ahcm
Copy link
Author

ahcm commented Feb 1, 2024

Same with current git version 5f40b7e.
Both datasets always segfault after producing the same amount of output (138GB and 191GB).

@cjw85
Copy link
Contributor

cjw85 commented Feb 2, 2024

Can you please provide the full pore-c-py annotate command that you are running? I think it would be worthwhile running also samtools quickcheck on the input you are giving to the program.

I'm unsure why you are setting NUMEXPR_MAX_THREADS. The numexpr Python package is not a direct dependency of pore-c-py (nor does it look like a transitive dependency after my quick check). Note also that the --threads parameter is used only to control the BAM compression threads used by htslib (through pysam). Setting the value to more than a handful is unlikely to result in linear performance gains.

@ahcm
Copy link
Author

ahcm commented Feb 2, 2024

samtools quickcheck shows nothing.

Without NUMEXPR_MAX_THREADS set, it complains about it being not set (and set it to a default).

srun -c 64 pore-c-py annotate "${INPUT}-${ENZYME}.bam.fastq-minimap-digest-ref.sam" "${OUTPUT}" --monomers --stdout --summary --chromunity

@ahcm
Copy link
Author

ahcm commented Feb 2, 2024

Do I even need to run annotate for yahs?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants