Closed davidanthoff closed 1 year ago
Patch coverage: 100.00
% and no project coverage change.
Comparison is base (
6ea9074
) 84.83% compared to head (7e4b216
) 84.83%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Yes, that deepcopy
was the problem for performance. If you approve, I'll merge.
There are two commits here that are unrelated.
The one about the
deepcopy
I'm especially not 100% sure. But I think I followed all code paths, and I think previously the deep copy that was created originally was then actually never (except for the one case where I added adeepcopy
returned to user, but was essentially always a temporary copy, in which case it seems unnecessary to me.