-
Notifications
You must be signed in to change notification settings - Fork 2
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
--read-likelihood crashes #37
Comments
Thanks for the report! This happens regardless of the size of the input, right? |
jnalanko
added a commit
to jnalanko/mSWEEP
that referenced
this issue
Oct 16, 2024
Not sure, but I found a way to patch this (see pull request). |
(That pull request also fixes another uninitialized pointer which was causing a crash.) |
tmaklin
added a commit
that referenced
this issue
Oct 16, 2024
Initialize pointers (fixes issue #37)
great, this is now fixed in v2.2.1 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
I'm trying to run mSWEEP with precomputed log-likelihoods. It is crashing here:
The reason for the crash is that
log_likelihoods
is a null pointer. It seems that it never gets initialized because it's initialized only when computing the likelihood matrix, which doesn't happen since I'm using pre-computed likelihoods. I think it should be initialized to some empty initial value in order to callfrom_file
, but I'm not sure which likelihood template should be used here, so I could not fix it myself yet.The text was updated successfully, but these errors were encountered: