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

Remove experimental warning logging from extract [#68] #92

Merged
merged 1 commit into from
Oct 19, 2023
Merged

Remove experimental warning logging from extract [#68] #92

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 0 additions & 4 deletions pmtiles/extract.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -251,8 +251,6 @@ func MergeRanges(ranges []SrcDstRange, overfetch float32) (*list.List, uint64) {

func Extract(logger *log.Logger, bucketURL string, key string, maxzoom int8, region_file string, bbox string, output string, download_threads int, overfetch float32, dry_run bool) error {
// 1. fetch the header

fmt.Println("WARNING: extract is an experimental feature and results may not be suitable for production use.")
start := time.Now()
ctx := context.Background()

Expand Down Expand Up @@ -555,8 +553,6 @@ func Extract(logger *log.Logger, bucketURL string, key string, maxzoom int8, reg
total_requests += num_overfetch_ranges
fmt.Printf("Extract required %d total requests.\n", total_requests)
fmt.Printf("Extract transferred %s (overfetch %v) for an archive size of %s\n", humanize.Bytes(total_bytes), overfetch, humanize.Bytes(total_actual_bytes))
fmt.Println("Verify your extract is usable at https://protomaps.github.io/PMTiles/")
fmt.Println("Feedback wanted! report your success or failure to https://github.com/protomaps/go-pmtiles/issues")

return nil
}