Arbitrary topoplot types

[asinghvi17]

Would it make sense to add a plottype kwarg to TopoPlot, which would allow any HeatmapLike plot to be used in place of contourf? I would love to use surface or even contour, for example.

[behinger]

maybe there is a good way. We already have an ugly if / else solution for "mesh" here:

TopoPlots.jl/src/core-recipe.jl

Line 104 in c3a56f7

if p.interpolation[] isa DelaunayMesh

So if this can be generalized, that would be great anyway.

But then I guess only replacing the surface command with a plottype would be much easier and more "surgical". I don't see the downside

[palday]

I don't have an objection to this, though I would do something more like plotfnc=heatmap! as a default attribute in the recipe. Then you just change

TopoPlots.jl/src/core-recipe.jl

Line 120 in 2523bd2

heatmap!(p, xg, yg, data, colormap=p.colormap, colorrange=colorrange, interpolate=true)

to use plotfnc instead of heatmap!.