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

Debug mode for Visual Studio solution is broken and several exceptions/crashes #61

Open
cbielow opened this issue Jul 2, 2024 · 1 comment

Comments

@cbielow
Copy link

cbielow commented Jul 2, 2024

When opening the Comet.sln in VS2019, compiling for Release works, but Debug config is out of date.
Its missing some include directories (cannot find "expat.h" etc) and some precursor defs (to disable sqlite, which is not included in the sources).

When running Comet.exe on Windows, several issues prevent a search to run to the end, e.g. (not necessarily in this order)

  1. a stack overflow (not sure where that came from, VS won't give a proper trace).
  2. some issues in parameter handling, e.g. when param.txt contains lines like:
peff_obo =
precursor_NL_ions =

which may not be valid param.txt, but Comet should not throw exceptions; in Release mode, this is also undefined behaviour and may have weird consequences, but luckily nothing happens, so far.

  1. A crash in this line (invalid handle) https://github.com/UWPR/Comet/blob/v2024.01.0/CometSearch/ThreadPool.h#L217 (does not happen in Release, but maybe it's just ignored).
@jke000
Copy link
Collaborator

jke000 commented Jul 2, 2024

I had ignored the broken Debug config for ages but just commited a fix to address that as well as the parameter handling so thanks for reporting both. I am seeing a crash in the Debug binary but at a different point than you noted. I'm currently still clueless as to why it dies where it does, upon entering CometSearch::RunSearch().

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