parcels icon indicating copy to clipboard operation
parcels copied to clipboard

Remove GridSet class from Public API

Open VeckoTheGecko opened this issue 5 months ago • 1 comments

I don't think the GridSet object really needs to be called by users at all, as they are not manually defining gridsets. All the grid information is available to the users via the Field.grid attributes, and FieldSet.gridset attributes.

I also think that the GridSet itself can be majorly simplified (ie, add_grid() removed/majorly refactored, grids being an actual set, turning it into a dataclass) to the point that we could maybe even outright replace it with some utility functions/processors that pull grid information from the Fields. That's out of topic for this issue, just a hint to where I can see this going.

@erikvansebille any insight onto the history of the gridset class?

VeckoTheGecko avatar Sep 10 '24 15:09 VeckoTheGecko