Rasters.jl icon indicating copy to clipboard operation
Rasters.jl copied to clipboard

cellarea API

Open alex-s-gardner opened this issue 6 months ago • 6 comments

The cellarea function call seems a bit bespoke for Rasters:

Should

Rasters.cellarea(Planar(), ras)

be changed to

Rasters.cellarea(ras; method = :planar)

alex-s-gardner avatar May 08 '25 01:05 alex-s-gardner

I'd say it's the opposite... this will be the GO syntax as well, it already is for segmentize

asinghvi17 avatar May 08 '25 01:05 asinghvi17

The syntax doesn't seem consistent with Rasters which relies heavily on kwargs

alex-s-gardner avatar May 08 '25 02:05 alex-s-gardner

I agree it doesn't really fit with other Rasters syntax

rafaqz avatar May 08 '25 13:05 rafaqz

But if GO is going to work this way, then it'd be weird if Rasters didn't

tiemvanderdeure avatar May 08 '25 15:05 tiemvanderdeure

Maybe, but consistent syntax here is important too

rafaqz avatar May 08 '25 15:05 rafaqz

Let's discuss this at some point, maybe I can also prototype some things for GO to unify

asinghvi17 avatar May 08 '25 15:05 asinghvi17