Unnecessary heap allocation?

[himanshugupta1009]

This line

GridInterpolations.jl/src/GridInterpolations.jl

Line 231 in fa8cd12

return grid.index[1:l]::Vector{Int}, grid.weight[1:l]::Vector{Float64}

allocates new vectors.

It seems like over-optimization to save a few Float computations at the expense of allocations which stresses the garbage collector. Is there a way to get around it? If not, is it possible to just avoid the extra check performed at line 229?

Thanks!

[mykelk]

Thanks for the idea! Feel free to contribute a PR with some empirical results on timing.

[himanshugupta1009]

I have created a PR for this issue with some empirical results. If there are any issues with it, let me know. Thanks!