Skip to content
This repository has been archived by the owner on Jul 3, 2023. It is now read-only.

Adds GeoPandas dataframe support #305

Merged
merged 1 commit into from
Feb 9, 2023
Merged

Adds GeoPandas dataframe support #305

merged 1 commit into from
Feb 9, 2023

Conversation

skrawcz
Copy link
Collaborator

@skrawcz skrawcz commented Feb 8, 2023

We missed adding this when we removed the "pandas" like dataframe assumption.

Changes

  • Adds geopandas plugin

How I tested this

  • Runs locally on something that uses geopandas

Notes

Checklist

  • PR has an informative and human-readable title (this will be pulled into the release notes)
  • Changes are limited to a single goal (no scope creep)
  • Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future TODOs are captured in comments
  • Project documentation has been updated if adding/changing functionality.

We missed adding this when we removed the "pandas" like dataframe
assumption.
Copy link
Collaborator

@elijahbenizzy elijahbenizzy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@skrawcz skrawcz merged commit 7fe662a into main Feb 9, 2023
@skrawcz skrawcz deleted the add_geopandas branch February 9, 2023 00:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants