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

Downloading XLSX sometimes has large column gaps #47

Open
longhotsummer opened this issue Aug 24, 2016 · 8 comments
Open

Downloading XLSX sometimes has large column gaps #47

longhotsummer opened this issue Aug 24, 2016 · 8 comments

Comments

@longhotsummer
Copy link
Collaborator

Downloading some wide XLSX data sometimes produces large empty columns. This seems to be an issue with the gdal library we're using to build the XLSX.

@longhotsummer
Copy link
Collaborator Author

It looks like this was fixed in GDAL 1.11 at some point a while back, I'm not sure why we're still seeing it.

https://trac.osgeo.org/gdal/ticket/5774

@longhotsummer
Copy link
Collaborator Author

I can confirm it still happens with GDAL 1.11.1, perhaps 1.11.2 fixed it.

@longhotsummer
Copy link
Collaborator Author

I think we should explore upgrading to GDAL 2.x, taking into account common GDAL versions on Ubuntu (14.04 and 16.04).

@longhotsummer
Copy link
Collaborator Author

Ubuntu 14.04 LTS (Trusty Tar) gets 1.10 by default, which is actually too old for Wazimap in any case, and we have to pull the 1.11 version from a separate repo.

Ubuntu 16.04 LTS has 1.11.3 which should be suitable.

So sticking with >1.11.1 and <2.0 seems reasonable for now.

https://launchpad.net/ubuntu/+source/gdal

@longhotsummer
Copy link
Collaborator Author

The buildpack we use to pull in this dependency in production (dokku, heroku) is fixed at 1.11.1 for the heroku/dokku ceder-14 stack: https://github.com/cyberdelia/heroku-geo-buildpack

$ curl http://cyberdelia-geo-buildpack.s3.amazonaws.com/cedar-14/manifest.gdal
1.11.1

@longhotsummer
Copy link
Collaborator Author

Opened cyberdelia/heroku-geo-buildpack#42

@longhotsummer
Copy link
Collaborator Author

Another alternative is to write our own Excel files, that's pretty straight forward.

@longhotsummer
Copy link
Collaborator Author

We may be able to use the Apt buildpack from heroku to pull in exactly the gdal we want. https://elements.heroku.com/buildpacks/heroku/heroku-buildpack-apt

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

No branches or pull requests

1 participant