Add `_sum` for `KernelSum{<:AbstractVector}` and associated tests

JuliaGaussianProcesses / KernelFunctions.jl

Julia package for kernel functions for machine learning

https://juliagaussianprocesses.github.io/KernelFunctions.jl/stable/

MIT License

267 stars 32 forks source link

Add `_sum` for `KernelSum{<:AbstractVector}` and associated tests #547

Closed theogf closed 6 months ago

theogf commented 6 months ago

Summary Add the _sum interface for vectors, allowing to use Vector in KernelSum. This is to fix #497

Proposed changes

Add a method for _sum and add the same tests existing for KerrnelSum{<:Tuple} to KernelSum{<:AbstractVector}.

What alternatives have you considered? :shrug:

Breaking changes none

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (49049b1) 90.88% compared to head (42316d9) 90.89%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #547 +/- ## ======================================= Coverage 90.88% 90.89% ======================================= Files 52 52 Lines 1459 1460 +1 ======================================= + Hits 1326 1327 +1 Misses 133 133 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.