agentscript0 icon indicating copy to clipboard operation
agentscript0 copied to clipboard

Model variable names

Open bennlich opened this issue 11 years ago • 1 comments

I'm thinking some of the model options variables could use a name change. Size should probably be patchSize, and minX/maxX/minY/maxY should be leftX/rightX/bottomY/topY?

bennlich avatar Aug 19 '14 19:08 bennlich

Definitely something other than "size". Possibly cellSize like GIS? .. then applies to both patches and datasets. But then patchSize is more explicit.

But left/right/top/bottom suggests left can be a larger number than right?

But indeed, readability matters a lot. No, a LOT! We could enforce left <= right. And yup, I do remember trying to sort it all out.

Its another NetLogo convention.

-- Owen

On Tue, Aug 19, 2014 at 1:32 PM, Benny Lichtner [email protected] wrote:

I'm thinking some of the model options variables could use a name change. Size should probably be patchSize, and minX/maxX/minY/maxY should be leftX/rightX/bottomY/topY?

— Reply to this email directly or view it on GitHub https://github.com/backspaces/agentscript/issues/56.

backspaces avatar Aug 19 '14 21:08 backspaces