Skip to content
This repository has been archived by the owner on Sep 4, 2024. It is now read-only.

Data wrangler refactor #197

Merged
merged 28 commits into from
Apr 1, 2022
Merged

Data wrangler refactor #197

merged 28 commits into from
Apr 1, 2022

Conversation

cjgrady
Copy link
Contributor

@cjgrady cjgrady commented Mar 29, 2022

Pull Request Type

  • Bug Fix
  • New Feature
  • Documentation
  • Other

Status

  • Ready
  • In development
  • Hold

Description

Refactor data wranglers to be more generic and classes. The wranglers are designed to be extended but have base functionality for building from a configuration dictionary (for the factory) and to have built-in reporting. Convert all of the old data wrangler functions to classes and add tests.

Link(s) to issue

#168

Todos

  • Tests
  • Documentation
  • Sample Data
  • Build Sphinx Documentation

Added dependencies

This probably requires that we update our GDAL required version to 3+ because of OSGeo/gdal#1546. We would need to add some specific handling for either GDAL 2 or change to GDAL 2 and handle GDAL 3 but GDAL 2 hasn't had an update for 2+ years.

@cjgrady cjgrady merged commit e1bd535 into main Apr 1, 2022
@cjgrady cjgrady deleted the wrangler-factory branch April 1, 2022 18:03
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.

Refactor data wranglers and make a common factory Something seems to be wrong with coordinate conversion
1 participant