Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix GDAL in continuous integration #1

Open
DentonGentry opened this issue Feb 8, 2020 · 1 comment
Open

Fix GDAL in continuous integration #1

DentonGentry opened this issue Feb 8, 2020 · 1 comment
Labels
good first issue Good for newcomers

Comments

@DentonGentry
Copy link
Collaborator

DentonGentry commented Feb 8, 2020

There is a continuous integration test defined in .github/workflows but it has never worked because the container needs to have GDAL installed. This needs to have some more cycles spent on it, to figure out how to adjust the workflow to install gdal before trying to run the tests.

Tests do pass when run locally.

@sashkaw
Copy link

sashkaw commented Nov 19, 2022

Hi @DentonGentry ,

I am not sure if this repo is still active, but I took a stab at adding gdal to the github workflow and submitted a pull request.

Note: The tests currently fail when running the workflow (not sure if I should open a new issue?). The first error that I get is for test_extract_country_data.test_kg shown below:

E       AttributeError: 'NoneType' object has no attribute 'GetRasterBand'

The tests also fail when I run them locally. The error seems to come from trying to open one of the .tif files in the repo. I was also unable to open the file in QGIS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants