GAA-UAM / scikit-fda

Functional Data Analysis Python package
https://fda.readthedocs.io
BSD 3-Clause "New" or "Revised" License
308 stars 58 forks source link

RKHS inner product #550

Open m5signorini opened 1 year ago

m5signorini commented 1 year ago

This PR adds the RKHS inner product.

Cases regarding the product between FDataBasis objects with different basis or with the covariance function not already expressed in the tensor basis, might be better to not consider them.

codecov[bot] commented 1 year ago

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (7488932) 86.67% compared to head (5c30710) 86.14%. Report is 1 commits behind head on develop.

:exclamation: Current head 5c30710 differs from pull request most recent head 5655385. Consider uploading reports for the commit 5655385 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #550 +/- ## =========================================== - Coverage 86.67% 86.14% -0.54% =========================================== Files 153 149 -4 Lines 12390 11783 -607 =========================================== - Hits 10739 10150 -589 + Misses 1651 1633 -18 ``` | [Files](https://app.codecov.io/gh/GAA-UAM/scikit-fda/pull/550?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=GAA-UAM) | Coverage Δ | | |---|---|---| | [skfda/misc/covariances.py](https://app.codecov.io/gh/GAA-UAM/scikit-fda/pull/550?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=GAA-UAM#diff-c2tmZGEvbWlzYy9jb3ZhcmlhbmNlcy5weQ==) | `84.54% <100.00%> (+0.14%)` | :arrow_up: | | [skfda/tests/test\_rkhs\_inner\_product.py](https://app.codecov.io/gh/GAA-UAM/scikit-fda/pull/550?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=GAA-UAM#diff-c2tmZGEvdGVzdHMvdGVzdF9ya2hzX2lubmVyX3Byb2R1Y3QucHk=) | `100.00% <100.00%> (ø)` | | | [skfda/misc/rkhs\_product.py](https://app.codecov.io/gh/GAA-UAM/scikit-fda/pull/550?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=GAA-UAM#diff-c2tmZGEvbWlzYy9ya2hzX3Byb2R1Y3QucHk=) | `89.28% <89.28%> (ø)` | | ... and [37 files with indirect coverage changes](https://app.codecov.io/gh/GAA-UAM/scikit-fda/pull/550/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=GAA-UAM)

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

m5signorini commented 1 year ago

Current test errors occur in test_pandas_fdatabasis and test_pandas_fdatagrid . They say that the fixture all_boolean_reductions can not found.

vnmabus commented 1 year ago

@m5signorini Maybe I wasn't clear. I was asking if there is anything else that you wanted to include, as the PR is marked as a draft, or if I have your permission to merge it.

m5signorini commented 1 year ago

Yes sorry, forgot to change it from draft. I think it can be merged.

vnmabus commented 1 year ago

Thank you for your answer! However, a test is failing (and the difference between the expected and actual result is too high). Can you take a look at it?