DOI-USGS / DOI-USGS/plio

Control Network Object should default to GeoPandas Object

Open
#109 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
16
Forks
26
PR merge metrics
No merged PRs in 30d

Description

We already have pandas and gdal as dependencies. We should just return the dataframe from an isis control network as a geopandas data frame. This avoids clients having to do stuff like:

```
gdf = gpd.GeoDataFrame(cnet, geometry=geopandas.points_from_xy(lons, lats))
```

This is a convenience item I believe, but will impact how we write out networks as well. We need to address #108 to confirm everything is working before adding this enhancement.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing issue #108 and the current ISIS control-network object and its dataframe conversion; the payload names no files or tests. Trace the existing network write-out path before deciding how a GeoPandas dataframe should flow through it. Done means the default returned object is GeoPandas-compatible and network writing still works.

Written by the indexing model from the issue text.

Assessment

Tech stack
pandas, python
Domain
data
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.