Repository navigation
Draw large polygon sets as raster tiles, with the Boston sample - #17
Merged
Merged
Conversation
Same data and behaviour as the React and Android samples: - Downtown: the bundled Overture extract, three nested areas. - City buildings / city parcels: the whole city, about 100,000 polygons and 1.4 million vertices each, read from the app's Documents folder (too large for the bundle; copy them there with devicectl, see BostonData.swift). Parcels are coloured by land use, with a legend. - A loading overlay with progress, an InfoBubble at the tap (parcel id and land use, or building name, height and id, and the vertex count), and a tap on the map elsewhere closes it. The tapped polygon is highlighted in place. The launch environment opens a given dataset and camera (MAPCONDUCTOR_SAMPLE_BOSTON_DATA / _AREA / MAPCONDUCTOR_SAMPLE_INIT_CAMERA), and MAPCONDUCTOR_SAMPLE_BOSTON_PROBE / MAPCONDUCTOR_SAMPLE_AUTOPAN print frame rates and drive pans for measurement. The tap label carries seq|id|lat|lng as its accessibility value, for UI tests.
…lop merged in Each submodule's feat/polygon-tiling now has its develop merged (overlay stacking, clear order, geodesic fill, hole union). Polygon tiles take the polygon band through OverlayStacking.rasterKey; the core's holesAreDisjoint uses the hole union's EdgeGrid.
# Conflicts: # ios-for-arcgis # ios-for-googlemaps # ios-for-here # ios-for-longdo # ios-for-mapbox # ios-for-mapkit # ios-for-maplibre # ios-for-mappls # ios-for-maptiler # ios-for-openmobilemaps # ios-for-tomtom # ios-sdk-core
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
何をするか
多数のポリゴン（既定: 2,000 個以上または頂点 50,000 以上）を、ネイティブの図形ではなくラスタータイルとして描きます（
PolygonTilingOptions/PolygonTiler）。web・Android・iOS で同じ設計です。このリポジトリで変えたこと
feat/polygon-tiling: ボストンの建物のサンプル（Polygons で 10 万件）。gitlink を ios-sdk-core / 各プロバイダの新しい先頭に向けました（develop は取り込み済み）。develop の取り込み
このブランチは push 済みだったので、rebase せず
origin/developを merge commit で取り込みました。xcodebuild test（シミュレータ）: 406 件 pass（develop の穴の結合の fixture を含む）。scripts/api-surface.sh check: 変えたモジュールは全部 ok、ベースラインの更新は不要。マージの順番
この機能は複数のリポジトリにまたがります。ios-sdk-core → 各 ios-for- → ios-sdk（親）* の順にマージが要ります。
この PR は大きな機能なので、マージするかはレビューのうえで判断してください。
先にマージが要る submodule の PR