Closed henry2004y closed 8 months ago
Attention: 2 lines
in your changes are missing coverage. Please review.
Comparison is base (
415a91f
) 52.16% compared to head (331cb94
) 51.76%.
Files | Patch % | Lines |
---|---|---|
src/TestParticle.jl | 92.30% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I don't quite understand the changes here. It doesn't seem to be any different from before the modifications.
The goal is to keep it the same while refactoring the logic. Additionally, if for some reasons the user wants to use getinterp
directly, it would be easier.
For handling the analytical field and numerical field consistently in
prepare
, I would suggest moving theField
construction outisde ofgetinterp
for the numerical field.