-
Notifications
You must be signed in to change notification settings - Fork 12
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
Migration path from v1.0.2.1 to v1.9.4.1 #31
Comments
For reference, the python docs for version 1.0 were very complete - is there something equivalent for v1.9? https://www.cs.helsinki.fi/group/pssmfind/doc/python/MOODS.html |
The documentation is on Github wiki: https://github.com/jhkorhonen/MOODS/wiki See also the |
Yes I had found the documentation - a lot of sections are 'in construction' however, and there's no clear python API described, so I had to go have a look at the C++ code to see what parameters the functions could take. Thanks for linking the wiki directly - it would help to link https://github.com/jhkorhonen/MOODS/wiki/Function-reference directly from the readme on github, right now there's only links to 'getting started' and 'installation and I didn't see this section. As for the both_strands scanning, how was 1.0 handling it? Just reverse complementing matrices internally, or somehow scanning both strands at once? How is performance of 1.9 vs 1.0 in that case (would be the same, or slower?). Also it would have been nice if no API breaking occurred - I have no problem about functions now being in submodules, but losing parameters / functionality will make this upgrade a bit more complicated than I anticipated (regarding the loss of both_strands=True as a parameter). |
Just to be clear where my confusion is coming from - issues such as #29 seemingly implying there is integrated reverse complement search functionality, and from the basic usage examples: https://github.com/jhkorhonen/MOODS/blob/master/python/scripts/ex-basic-usage.py #separate reverse complements and the non-reverse complements #mix the results together, use + and - to indicate strand |
Ok had a look at 1.0 code, both_strands is nothing fancy so I can write a wrapper to do the same - still puzzled why this was left out as an option since its so simple / handy. Also puzzled by the inconsistent docs (I think there's a mix of information between versions). |
Yes, the complement strand scanning is just adding the reverse complement matrices to the matrix set, and this was the implementation in 1.0 also. The The 1.0 API was a mess, and doing some API breaking was necessary anyways, so I decided to just rip off the band-aid and make a more sane API from ground up back when I did the big 1.9 update. The lack of Sadly I've since moved away from bioinformatics, so finishing the documents and other MOODS stuff gets buried under everything else very easily... |
No worries, I got everything I needed from looking at the code :) Great library with amazing performance! |
I have old scripts using MOODS that I need to update and noticed that the API for MOODS changed significantly between v1 and v1.9. The documentation for the new version is however almost non-existant.
I was using only 2 functions:
MOODS.max_score()
and
MOODS.search(sequence, matrix_list, threshold_list, q=q, bg=bg, absolute_threshold=True, both_strands=True)
What would be the equivalent calls for v1.9?
The text was updated successfully, but these errors were encountered: