Open ioannisPApapadopoulos opened 1 year ago
Patch coverage: 95.23
% and project coverage change: -0.05
:warning:
Comparison is base (
02b1216
) 96.21% compared to head (b80a9a4
) 96.17%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Windows tests are failing but that seems to be the case in the main branch as well. Not sure why the code coverage has decreased.. I'm covering the lines that it's complaining about in the tests.
I've noticed in other packages that the macro @simplify
screws with coverage. I don't know why but as far as I've seen those lines never satisfy the coverage checks.
I managed to implement the partial derivatives with respect to y. The following code now runs:
`using MultivariateOrthogonalPolynomials
W = Weighted(Zernike(1)) Δ = Laplacian(axes(W,1))
∂ʸ = PartialDerivative{2}(axes(W,1))
Z⁰ = Zernike(0) Z¹ = Zernike(1)
∂Y² = (Z¹ \ (∂ʸ Z⁰)) (Z⁰ \ (∂ʸ W)) Δ = Z¹ \ (Laplacian(axes(W,1)) W)
z=2.0; A = (∂Y² - z*Δ); c = A[Block.(1:50), Block.(1:50)] \ rand(1275)`