-
Notifications
You must be signed in to change notification settings - Fork 6
New algorithm #16
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
base: master
Are you sure you want to change the base?
New algorithm #16
Conversation
The codes were directly translated from the original matlab codes. I haven't studied the algorithm or done any Julia localization. |
Thanks for the contribution! I will review it. |
This looks good. Even thought it's translated, it does not look like unidiomatic Julia for the most part. Does it work? It could use some kind of test. It looks like Also, if this is better than the existing code in all regimes, then of course it could replace the old implementation/algorithm. |
The only problem I can see is that the default tolerance is set to I didn't add any test since I was not sure how you would manage the codes. I'm not exactly sure when |
I find that this routine is ten times slower than the current one for some parameters julia> @btime mittleff_matlab(0.3, 1.0, .4)
11.311 μs (96 allocations: 10.83 KiB)
1.725742301414842
julia> @btime mittleff(0.3, 1.0, .4)
1.408 μs (0 allocations: 0 bytes)
1.7257423014148419 and similar for other parameters. But I don't doubt that behavior in various regions of the three-parameters space is complicated. I think it's ok to commit this with a few changes
There are a few things that could be tried to improve efficiency of the "matlab" version. But the current state of the code is rather crude and not optimized. So I would not hold this up based on efficiency. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #16 +/- ##
===========================================
- Coverage 77.30% 36.82% -40.49%
===========================================
Files 2 3 +1
Lines 141 296 +155
===========================================
Hits 109 109
- Misses 32 187 +155 ☔ View full report in Codecov by Sentry. |
https://github.com/jlapeyre/MittagLeffler.jl/issues/15