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

Import broken, XYZ never implemented #34

Merged
merged 1 commit into from Aug 10, 2012
Merged

Conversation

klokan
Copy link
Contributor

@klokan klokan commented Aug 10, 2012

As reported at http://support.mapbox.com/discussions/general-questions/2682-mb-util-exportimport-no-such-file-error
the import of exported mbtiles does not run.

It is possible to import only if there is no version/name prefix such as "1.0.0/layer-name" and only TMS tiles.
Right now default for export is "XYZ".

Solution:

  • Implemented flip_y() call for XYZ scheme (which is now default) during importing
  • The "version/name" subdirectory prefix has been removed during the export. I expect it is there only for dummy WMTS implementations (where it is not required by the standard anyway). It does not hold any information. Directory with metadata.json can anyway only contain one tileset.

Now import and export of the same .mbtiles runs (beside grids were not implemented before either) for both TMS and XYZ scheme.

tmcw added a commit that referenced this pull request Aug 10, 2012
Import broken, XYZ never implemented
@tmcw tmcw merged commit 73772d8 into mapbox:master Aug 10, 2012
@klokan
Copy link
Contributor Author

klokan commented Aug 10, 2012

It would be great to make also new release of mb-util on http://pypi.python.org/pypi, because this is quite an important bug.

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

Successfully merging this pull request may close these issues.

None yet

2 participants