go-spatial / tegola

45%

master: 40%

LAST BUILD ON BRANCH issue-224-mvt-refactor

branch: issue-224-mvt-refactor

CHANGE BRANCH

x

Reset

• issue-224-mvt-refactor
• Hoovs-read_only
• add-dont_simplify-syntax
• add-govulncheck-to-ci
• add_README_for_prometheus_observer
• add_ability_to_specify_grid_for_cache_cmd
• add_cache_observer
• add_http_end_point_observer
• assetfs-cleanup
• bbox-seeding-no-dups
• bug_in_dp
• bump-go-1.22
• bump-go-version
• bump-go-version-1.21.8
• cache-seed
• changelog-update
• chore/bump-pgx-v5
• ci-updates
• ci/dont-publish-edge-if-no-auth
• ci/use-docker-compose
• ci_install_geom
• cobra-rollback
• code-review-issue-159-viewer-multi-map-support
• config-stdin
• contributing_guide
• data_provider_interface_refactor
• dependabot
• dependabot/go_modules/github.com/aws/aws-sdk-go-1.33.0
• dependabot/go_modules/github.com/aws/aws-sdk-go-1.34.0
• dependabot/go_modules/github.com/jackc/pgx/v4-4.18.2
• dependabot/go_modules/github.com/prometheus/client_golang-1.11.1
• dependabot/go_modules/github.com/redis/go-redis/v9-9.7.3
• dependabot/go_modules/golang.org/x/crypto-0.1.0
• dependabot/go_modules/golang.org/x/crypto-0.17.0
• dependabot/go_modules/golang.org/x/crypto-0.31.0
• dependabot/go_modules/golang.org/x/net-0.17.0
• dependabot/go_modules/golang.org/x/net-0.36.0
• dependabot/go_modules/golang.org/x/net-0.38.0
• dependabot/go_modules/golang.org/x/net-0.7.0
• dependabot/go_modules/golang.org/x/oauth2-0.27.0
• dependabot/go_modules/google.golang.org/grpc-1.53.0
• dependabot/go_modules/google.golang.org/grpc-1.56.3
• dependabot/go_modules/google.golang.org/protobuf-1.33.0
• dependabot/npm_and_yarn/ui/ajv-6.12.6
• dependabot/npm_and_yarn/ui/async-2.6.4
• dependabot/npm_and_yarn/ui/axios-0.21.1
• dependabot/npm_and_yarn/ui/axios-0.21.2
• dependabot/npm_and_yarn/ui/axios-1.6.0
• dependabot/npm_and_yarn/ui/axios-1.7.4
• dependabot/npm_and_yarn/ui/axios-1.8.2
• dependabot/npm_and_yarn/ui/babel/traverse-7.23.9
• dependabot/npm_and_yarn/ui/browserslist-4.16.6
• dependabot/npm_and_yarn/ui/decode-uri-component-0.2.2
• dependabot/npm_and_yarn/ui/dns-packet-1.3.4
• dependabot/npm_and_yarn/ui/elliptic-6.5.4
• dependabot/npm_and_yarn/ui/eventsource-1.1.1
• dependabot/npm_and_yarn/ui/follow-redirects-1.14.7
• dependabot/npm_and_yarn/ui/follow-redirects-1.14.8
• dependabot/npm_and_yarn/ui/follow-redirects-1.15.5
• dependabot/npm_and_yarn/ui/follow-redirects-1.15.6
• dependabot/npm_and_yarn/ui/form-data-4.0.4
• dependabot/npm_and_yarn/ui/highlight.js-9.18.5
• dependabot/npm_and_yarn/ui/hosted-git-info-2.8.9
• dependabot/npm_and_yarn/ui/json5-and-json5-and-vue/cli-service-2.2.3
• dependabot/npm_and_yarn/ui/json5-and-vue/cli-plugin-babel-and-vue/cli-plugin-eslint-and-vue/cli-service-2.2.3
• dependabot/npm_and_yarn/ui/lodash-4.17.19
• dependabot/npm_and_yarn/ui/lodash-4.17.21
• dependabot/npm_and_yarn/ui/minimist-1.2.6
• dependabot/npm_and_yarn/ui/multi-78801bc13d
• dependabot/npm_and_yarn/ui/multi-8d5dbb2794
• dependabot/npm_and_yarn/ui/nanoid-3.3.8
• dependabot/npm_and_yarn/ui/path-parse-1.0.7
• dependabot/npm_and_yarn/ui/postcss-7.0.36
• dependabot/npm_and_yarn/ui/postcss-8.4.35
• dependabot/npm_and_yarn/ui/qs-6.5.3
• dependabot/npm_and_yarn/ui/rollup-3.29.5
• dependabot/npm_and_yarn/ui/shell-quote-1.7.3
• dependabot/npm_and_yarn/ui/ssri-6.0.2
• dependabot/npm_and_yarn/ui/terser-4.8.1
• dependabot/npm_and_yarn/ui/url-parse-1.5.10
• dependabot/npm_and_yarn/ui/url-parse-1.5.3
• dependabot/npm_and_yarn/ui/url-parse-1.5.7
• dependabot/npm_and_yarn/ui/vite-4.3.9
• dependabot/npm_and_yarn/ui/vite-4.5.10
• dependabot/npm_and_yarn/ui/vite-4.5.13
• dependabot/npm_and_yarn/ui/vite-4.5.14
• dependabot/npm_and_yarn/ui/vite-4.5.2
• dependabot/npm_and_yarn/ui/vite-4.5.3
• dependabot/npm_and_yarn/ui/vite-4.5.9
• dependabot/npm_and_yarn/ui/word-wrap-1.2.4
• dependabot/npm_and_yarn/ui/ws-6.2.2
• dependabot/npm_and_yarn/ui/y18n-4.0.1
• documentation-updates
• drop-bindata-requirement
• ear7h-issue-403
• encode-cleanup
• filterer_implementation
• filterer_interface
• fix-docker-hub-login
• fix-seed-bounds-srid-arg
• fix_prometheus_error_issue
• fix_ui_build
• fixed-log-messages
• gd0.6.44
• gd0.6.45
• gd0.6.46
• gd0.6.47
• gd0.6.48
• gd0.6.49
• geom-mvt-fix
• geom-spherical-pr
• geom-spherical-pr2
• gh-actions
• gh-actions-on-publish
• go-mod
• gofmt_code
• gpkg_autoconfig
• initial_prometheus
• integrate_geom_makevalid
• internal-viewer-build-fix
• issue-1000-bump-go-version
• issue-1022-enahce-hostName-handling
• issue-136
• issue-159-viewer-multi-map-support
• issue-224
• issue-338
• issue-361
• issue-375_empty_layer_should_return_404
• issue-379
• issue-383
• issue-388
• issue-420
• issue-425
• issue-429
• issue-432
• issue-435
• issue-437
• issue-438
• issue-445
• issue-468
• issue-480
• issue-484
• issue-489
• issue-490
• issue-494
• issue-514
• issue-519
• issue-538
• issue-553
• issue-573
• issue-586-file-cache-too-many-files-open-fix
• issue-590
• issue-594
• issue-609
• issue-622
• issue-626
• issue-628-2
• issue-633
• issue-698-cache-seed-zoom-filter-handling
• issue-728-Docker-failing
• issue-731-st_as_mvt_srid_4326
• issue-76_config_bind_and_port
• issue-789-tegola_lambda-glib-bug
• issue-799
• issue-805-gpkg-error
• issue-82
• issue-89
• issue-942-view-not-built
• issue-943-tegola-lambda-al2-support
• issue-994-malformed-hostnames
• issue-xx-upgrade-go-1.24.5
• issue-xx-viewer-enhancements
• issue_270_clean_up_boundbox
• issue_449
• issue_735
• issue_813
• mapl-enhancement-collapse-sidebar
• master
• migrate_mvt_providers
• mvt_provider_type_688
• nickelbob-issue-426-postgres-ssl
• normalize_table_tests
• observability_postgis_data_provider
• olt-postgis-sub-queries
• patch-1
• paumas-implement_dont_clip_flag
• pgx-numeric
• port_simplify
• postgis_autoconfig
• public_dicter
• quick_fix
• readme_bump_required_go_version
• refactor/slog
• refactor_build_info
• refs/tags/v0.10.4
• refs/tags/v0.11.0
• refs/tags/v0.11.1
• refs/tags/v0.11.2
• refs/tags/v0.12.0
• refs/tags/v0.12.1
• refs/tags/v0.13.0
• refs/tags/v0.14.0
• refs/tags/v0.15.0
• refs/tags/v0.15.1
• refs/tags/v0.15.2
• refs/tags/v0.16.0
• skip_s3_tests_when_no_creds
• slippy_tile_issue_997
• split_out_geom
• stvno-docker
• stvno-explicitTiles
• style_cleanup
• tegola_lambda_cgo
• tstv0.6.1
• ui-build-script-improvement
• ui-deps-updates
• update-geom
• update-gh-workflow
• update-readme
• update-sqlite
• update_code_owners
• upgrade-go
• upgrade-mapboxgl
• upgrade_pgx
• v0.10.0
• v0.10.1
• v0.10.2
• v0.10.3
• v0.10.x
• v0.11.x
• v0.11.x-new
• v0.11.x_dry_run
• v0.11.x_geom
• v0.11.x_stage
• v0.12.x
• v0.12.x-readme-updates
• v0.12.x_geom
• v0.12.x_geom_makevalid
• v0.12.x_stage
• v0.13.x
• v0.14.x
• v0.15.x
• v0.6.1
• v0.6.2
• v0.6.3
• v0.6.4
• v0.7.0
• v0.8.0
• v0.8.1
• v0.8.x
• v0.9.0
• v0.9.x
• vendor-cleanup
• vuln-squashing

Build # 1703

Build Type

push

travis-ci

Committed by ARolek

Commit Message

Move geometry processing out of mvt package

The mvt encoding package contained core geo processing routines
(i.e. simplification, make valid, scaling). These routines have
been carved out of the mvt pacakge and moved into the atlas
map Encode() step. This allows the mvt package to focus entirely
on encoding.

This is a transitional step which positions tegola to adopt the
new make valid routine which is currently under development in the
geom package. This transition also positions tegola to adopt the
new simplification routine which is in the geom package. The mvt
package will soon be carved out of tegola and also moved to the
geom package.

During the refactor, a fair amount of code was purged from the mvt
package and many function signatures were able to reduce the number
of parameters they required. Additionally the mvt feature
encoding tests were able to fixed to match the specifications
supplied tests and previously breaking tests are now passing.

closes #224

Run Details

193 of 193 new or added lines in 7 files covered. (100.0%)

5424 of 12055 relevant lines covered (44.99%)

827.17 hits per line