-
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
Preloop for constructors #51
Conversation
Codecov Report
@@ Coverage Diff @@
## master #51 +/- ##
==========================================
- Coverage 41.85% 39.40% -2.46%
==========================================
Files 8 8
Lines 571 571
==========================================
- Hits 239 225 -14
- Misses 332 346 +14
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
The implementation of the pre-loop is completed, should we merge into master? I guess some of the checks fail because of code syntax.. |
thank a lot, I'll take a look and merge it asap! |
I've implemented a preloop that computes constructors for all Markov scales in a parallelised way and stores them for later use. I hope this speeds up the process because computing the constructors is the bottleneck of MS analysis, which was previously not parallelised. The implementation seems to work but speed increase needs to be checked.