Skip to content

Point ios-sdk-core at the polygon hole union fix - #16

Merged
wf9a5m75 merged 2 commits into
developfrom
chore/bump-core-hole-union
Nov 9, 2026
Merged

wf9a5m75 merged 2 commits into
developfrom
chore/bump-core-hole-union

Conversation

@wf9a5m75

@wf9a5m75 wf9a5m75 commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

なぜ

近い穴を結合するときに穴の面積が失われ、街区などが塗りつぶされていた不具合の修正が ios-sdk-core でマージ済みです:

何を変えたか

  • gitlink: ios-sdk-core を上の develop のマージ commit(ca81984)に向けた。
  • ほかの 20 モジュールの gitlink も、それぞれの develop の先頭に向けました(gitlink 以外は変えていません)。前の core(8709b1a)から ca81984 までに、MarkerRenderingSupport が clearMarkers() を要求するようになっています(clear strategy, ios-sdk-core#30)。プロバイダ側の対応は各 develop にマージ済みですが、ここの gitlink が古いままで、core だけ進めるとサンプルがビルドできません(HereMapHost does not conform to protocol 'MarkerRenderingSupport')。
    • 進めたもの: ios-for-{arcgis, googlemaps, here, longdo, mapbox, mapkit, maplibre, mappls, maptiler, openmobilemaps, template, tomtom}、ios-{geojson, heatmap, icons, icons-jp, icons-us, icons-weather, kml, marker-clustering}。どれも fast-forward(今の gitlink を含む)。
    • これで core の重なり順(OverlayStacking, #31)・geodesic の閉じる辺(#29)・clear の順序(#27, #28)と、それに合わせたプロバイダの変更も入ります。

検証

  • ios-sdk-core の xcodebuild test(iPhone 17 Pro シミュレータ): 371 件、失敗 0。
  • scripts/api-surface.sh check ios-sdk-core: ok(CI と同じ範囲)。
  • サンプル(MapConductorSampleApp)の Debug ビルド(generic iOS): 成功。

確認できていないもの

  • 実機での表示(ビルドのみ)。穴の表示は core の PR #32 で iPad 実機確認済み。
  • 手元の scripts/api-surface.sh check(全モジュール)は、core 以外で落ちるものがあります。ArcGIS / Google / Mapbox / MapLibre / MapTiler / Mappls / icons は一時ディレクトリでのバイナリ取得(xcframework の Info.plist が無い)で、ios-geojson / ios-kml は型名の表記(UIKit.UIColor → UIKit::UIColor)の差分、heatmap / marker-clustering はベースライン未作成です。いずれもこの PR の変更(gitlink)による API の変化ではありません。

ios-sdk-core ca81984 (MapConductor/ios-sdk-core#32): unionHoleRings merges touching or overlapping holes so their shared area is not filled back in. Also brings in the core changes merged since 8709b1a (overlay stacking order, clear strategy, geodesic fill, clear order).
ios-sdk-core ca81984 asks MarkerRenderingSupport for clearMarkers() (clear strategy, ios-sdk-core#30); the providers' develops already have it (merged), the gitlinks here did not. Without this the sample does not build (HereMapHost does not conform).
@wf9a5m75
wf9a5m75 merged commit 4df5dc9 into develop Nov 9, 2026
1 check passed
@wf9a5m75
wf9a5m75 deleted the chore/bump-core-hole-union branch November 9, 2026 16:29
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.

1 participant