evetion/GeoDataFrames.jl

Change some args in `write` to be keyword?

vlandau opened this issue · 4 comments

Wondering if it might make it easier on the user to have some of the arguments to write be keywords?

Then the user wouldn't have to specify every argument if they only want to change, e.g., crs or driver.

Sure, do you propose to make all parameters except the first two keyword arguments?

visr commented

Clicking that link I'm mainly surprised by this default: crs::GFT.GeoFormat=GFT.EPSG(4326). Isn't no CRS better than wrong CRS?

Sure, do you propose to make all parameters except the first two keyword arguments?

Yep ☝️ that's exactly what I was thinking.

@visr I would agree that no CRS would be better, and hopefully this will become even less of an issue once projection info is stored inside the GeoDataFrame itself, which I believe is the plan.

The keywords are fixed in the latest release. The default projection isn't yet fixed, but is now an issue: #9.