-
Notifications
You must be signed in to change notification settings - Fork 1
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
Update to HiddenMarkovModels v0.6 #9
Conversation
Thanks! I just up version and added your new |
This is not documented so it is not part of the public API and could break in patch release. At the moment you should not rely on this functionality. |
The main reason I introduced it was to activate or deactivate multithreading based on dispatch, so that a single sequence (passed as |
Do you recommend removing the type annotation for |
I would recommend simply removing the annotation on |
And lastly, you don't need to release a breaking version when you update a dependency like that, except when it has an impact on your package's own user-facing API. So here tagging v0.2.1 would probably have sufficed. But we can discuss it face to face in Toulouse ;) |
Dam! I can always revert the pkg number since it not release yet? Can't wait (I'll be there only Tuesday, unfortunately)! |
I recently released a breaking version of HiddenMarkovModels, so here's a small PR updating the compat bound and adding a few
view
s in passing for speed