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.

Sign up for GitHub

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

Jump to bottom

Extract command error #52

Closed
andrewpmoore opened this issue May 8, 2023 · 3 comments
Closed

Extract command error #52

andrewpmoore opened this issue May 8, 2023 · 3 comments

Comments

@andrewpmoore
Copy link

I've downloaded the sample 2.4gb level 10 zoom map and wanted to try and make one that only zooms to level 8
I'm trying to run the command to extract like this on a windows command line

pmtiles.exe extract --maxzoom=8 z10.pmtiles z8.pmtiles

But I just get the error

panic: extract <input> <output>

goroutine 1 [running]:
main. Main()
        /Users/bdon/workspace/protomaps/go-pmtiles/main.go:166 +0xc1e

Is there anything I'm doing wrong there, note that /Users/bdon/workspace is not my directory location, so not sure where that's coming from
bdon added a commit that referenced this issue May 8, 2023
@bdon
add trimpath to suppress build machine paths in backtraces [#52]
d335f16
bdon added a commit that referenced this issue May 8, 2023
@bdon
fix panic: not enough args in extract [#52]
26fc088
@bdon
Copy link
Member

bdon commented May 8, 2023 •
edited
Loading

You're not doing anything wrong, it isn't implemented yet ( #31 ). Fixed the error message and also suppressed those paths in the debug symbols

@andrewpmoore
Copy link
Author

Thanks @bdon, any idea how your go about getting a global level 0 to 8 map or modifying the 0 to 10 file supplied?

bdon added a commit that referenced this issue Sep 4, 2023
@bdon
implement pmtiles extract [#31, #52]
27a426d 
* Experimental cli support for extracting a region from a larger archive, given a maxzoom and GeoJSON multipolygon region.
* Limited to credentialed buckets or local files now, public HTTP to come later
* Limited to a single download thread
* Change directory optimization to be faster and match Java implementation, affects root/leaf sizes
@bdon
Copy link
Member

bdon commented Sep 4, 2023

pmtiles extract as a experimental feature is being merged soon in #62

This should allow pmtiles extract input.pmtiles output.pmtiles --maxzoom 8 on a local file or remote storage bucket.

Closing this issue now, welcome to discuss in PR or #31

@bdon bdon closed this as completed Sep 4, 2023
bdon added a commit that referenced this issue Sep 4, 2023
@bdon
implement pmtiles extract [#31, #52] (#62)
9bc8c22 
* implement pmtiles extract [#31, #52]

* Experimental cli support for extracting a region from a larger archive, given a maxzoom and GeoJSON multipolygon region.
* Limited to credentialed buckets or local files now, public HTTP to come later
* Limited to a single download thread
* Change directory optimization to be faster and match Java implementation, affects root/leaf sizes

* Finish initial extract [#31]

* include the DstOffset so we can multithread downloads later
* set header statistics
* implement --dry-run
* add logging messages for user feedback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bdon @andrewpmoore