-
Notifications
You must be signed in to change notification settings - Fork 17
Upgrade TaylorSeries to v0.18 and TaylorIntegration to v0.16 #171
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
Conversation
Pull Request Test Coverage Report for Build 10232509876Details
💛 - Coveralls |
Pull Request Test Coverage Report for Build 10232509876Details
💛 - Coveralls |
Pull Request Test Coverage Report for Build 12995972981Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
The test that fails (only in Julia 1.6) is related to an ambiguity detected by Aqua; I'll ignore it for the time being... |
Mmmm, there seems to be some odity related to coveralls... |
I have no experience with coveralls (we always use codecov). The error message is indeed not helpful. Fixing the cause would of course be better, but maybe you can add the following line inside the TaylorModels.jl/.github/workflows/ci.yml Lines 52 to 55 in bfc5791
|
@schillic I think this is ready to be merged (and tagged). Could you check that everything works fine wrt ReachabilityAnalysis ? |
I got an error: pkg> st
[...]
[92b13dbe] TaylorIntegration v0.16.2
[314ce334] TaylorModels v0.7.1 `https://github.com/JuliaIntervals/TaylorModels.jl.git#lb/upgrade_TS`
⌅ [6aa5eb33] TaylorSeries v0.18.4
[...]
pkg> test
[...]
MethodError: no method matching _evaluate!(::TaylorN{IntervalArithmetic.Interval{Float64}}, ::HomogeneousPolynomial{IntervalArithmetic.Interval{Float64}}, ::Tuple{TaylorN{Float64}, TaylorN{Float64}}, ::Vector{TaylorN{Float64}}, ::TaylorN{IntervalArithmetic.Interval{Float64}})
The function `_evaluate!` exists, but no method is defined for this combination of argument types.
Closest candidates are:
_evaluate!(::TaylorN{T}, ::HomogeneousPolynomial{T}, ::Int64, ::TaylorN{T}, ::TaylorN{T}) where T<:Union{Real, Complex, Taylor1}
@ TaylorSeries ~/.julia/packages/TaylorSeries/Mdt5d/src/evaluate.jl:406
_evaluate!(::TaylorN{T}, ::HomogeneousPolynomial{T}, ::NTuple{N, var"#s485"} where var"#s485"<:TaylorN{T}, ::Array{TaylorN{T}, 1}, ::TaylorN{T}) where {N, T<:Union{Real, Complex}}
@ TaylorSeries ~/.julia/packages/TaylorSeries/Mdt5d/src/evaluate.jl:197
_evaluate!(::TaylorN{T}, ::NTuple{N, TaylorN{T}}, ::TaylorN{T}, ::Array{TaylorN{T}, 1}, ::TaylorN{T}) where {N, T<:Number}
@ TaylorSeries ~/.julia/packages/TaylorSeries/Mdt5d/src/evaluate.jl:501 |
TaylorIntegration = "0.15" | ||
TaylorSeries = "0.17" | ||
TaylorIntegration = "0.16" | ||
TaylorSeries = "0.18.4" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you have to go directly to TaylorSeries = "0.18.4"
or could you also go to TaylorSeries = "0.18"
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you need to upgrade to TS v0.18.4...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But I'm not completely sure
Do you need an input to debug or is it clear what causes the problem? |
I'll have a look to see if I can reproduce the problem... |
@schillic I can reproduce the error; can you tell me which was the last version (of TS) that allowed the tests to pass. Perhaps the reason for the problem is that I forgot to include a method that is precisely needed here. |
This branch only allows v0.18.4. |
I think I found the problem... Can you test again using for TS the branch in JuliaDiff/TaylorSeries.jl#378 ? (Locally, tests pass, but I prefer to have your confirmation.) |
@schillic I juste merged JuliaDiff/TaylorSeries.jl#378; bumping the patch version is on its way. Once that is done, I think I can merge this PR |
I was too slow debugging something else, but now I re-ran the tests with your branch and indeed they pass 👍 |
Thanks a lot Christian! I guess I have to tag a minor version, right? |
No description provided.