KML Sink:建物ごとにPlacemarkを作成するように変更 #278
Merged
Conversation
This file contains 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
Walkthroughこの変更は、 Changes
TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review Status
Actionable comments generated: 4
Configuration used: CodeRabbit UI
Files selected for processing (2)
- nusamai-kml/src/conversion.rs (4 hunks)
- nusamai/src/sink/kml/mod.rs (3 hunks)
Additional comments: 4
nusamai-kml/src/conversion.rs (4)
- 10-29:
multipolygon_to_kml、indexed_multipolygon_to_kml、およびmultipolygon_to_kml_with_mapping関数の変更は適切です。Vec<KmlPolygon>への変更は、建物ごとにPlacemarkを作成する目的に合致しています。- 31-43: 新しい関数
polygon_to_kml_polygon_with_mappingは、個々のポリゴンをKmlPolygonに変換するための適切なアプローチを提供します。この関数は、提案された変更の目的をサポートするために重要です。- 101-114:
polygon_to_kml_with_mappingおよびindexed_polygon_to_kml関数の変更も、Vec<KmlPolygon>への変更を反映しており、適切です。- 217-229: テストケース
test_polygon_to_kmlは、変更された関数の動作を適切に検証しています。ポリゴンの外部リングと内部リングの座標が正しく変換されていることを確認しています。
Codecov ReportAttention:
Additional details and impacted files
📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Review Status
Actionable comments generated: 4
Configuration used: CodeRabbit UI
Files selected for processing (1)
- nusamai-kml/src/conversion.rs (4 hunks)
Additional comments: 1
nusamai-kml/src/conversion.rs (1)
- 101-102:
polygon_to_kml_with_mapping関数で、戻り値をVec<KmlPolygon>に変更していますが、この関数は常に1つのKmlPolygonを含むベクターを返します。この設計は意図的なものでしょうか？もし単一のKmlPolygonのみを扱う場合は、戻り値の型をVec<KmlPolygon>ではなくKmlPolygonにすることを検討してください。
There was a problem hiding this comment.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (1)
- nusamai-kml/src/conversion.rs (5 hunks)
Additional comments: 9
nusamai-kml/src/conversion.rs (9)
- 10-10:
multipolygon_to_kml関数の変更は、戻り値の型をVec<KmlPolygon>に変更しています。この変更は目的に合致しています。- 17-17:
indexed_multipolygon_to_kml関数も同様に、戻り値の型をVec<KmlPolygon>に変更しています。適切な変更です。- 24-29:
multipolygon_to_kml_with_mapping関数の変更により、Vec<KmlPolygon>が戻り値となっています。この変更は、建物ごとにPlacemarkを作成する目的に沿っています。- 31-43: 新しく追加された
polygon_to_kml_polygon_with_mapping関数は、単一のポリゴンをKmlPolygonに変換します。この関数は、提案された変更の一環として適切に追加されています。- 101-102:
polygon_to_kml_with_mapping関数の戻り値の型がVec<KmlPolygon>に変更されています。この変更は、提案された目的に沿っています。- 106-107:
polygon_to_kml関数の戻り値の型がVec<KmlPolygon>に変更されています。この変更は、提案された目的に沿っています。- 111-115:
indexed_polygon_to_kml関数の戻り値の型がVec<KmlPolygon>に変更されています。この変更は、提案された目的に沿っています。- 106-106: 過去のレビューコメントで指摘された
poly.clone()の使用について、この変更では直接的な修正が見られません。不必要なクローン操作を避けるための改善が必要かもしれません。- 111-114: 過去のレビューコメントで指摘された
poly_idx.clone()の使用についても、この変更では直接的な修正が見られません。不必要なクローン操作を避けるための改善が必要かもしれません。
|
@xinmiaooo @coderabbitai |
|
@ciscorn このコミットの変更を確認しました。 Analysis chainScripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Fetch the diff of the specified commit to analyze the changes made.
git diff 3496603d598106dd105c09f137622c813d4a8fbe^ 3496603d598106dd105c09f137622c813d4a8fbe
Length of output: 3001 |
Merged
xinmiaooo
added a commit
that referenced
this pull request
Feb 15, 2024
#278 のマージが前提です。 属性をKML地物に付与し、Google Earth上で表示可能なSchemaを付与。 ExtendedDataなので、機械的にも可読である  <img width="1582" alt="スクリーンショット 2024-02-15 12 12 28" src="https://github.com/MIERUNE/nusamai/assets/83005951/4ab95d92-705c-4642-b247-12992ca83265"> TODO： - 属性の階層を処理していないので、文字列になっている <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **新機能** - `KmlPolygon`の`Vec<KmlPolygon>`への変更 - `polygon_to_kml_polygon_with_mapping`の導入 - 既存の関数の更新 - `CityObjects`から`GeoJSON`オブジェクトから`KML`オブジェクトへの切り替え - 新しいデータ構造の導入: `Element`, `Geometry`, `Polygon`, `SimpleData` - プロパティの処理と`KML`要素の生成ロジックの更新 - スキーマデータエントリの作成 - 拡張データの処理 - `KML`出力の構造の調整 - `.gitignore`に`data/*`と`*.kml`を追加して、これらのファイルとディレクトリをバージョン管理から除外 - `.vscode/launch.json`にLLDBを使用してプロジェクト内の異なるコンポーネントとテストをデバッグするための様々な起動構成を設定 - 実行可能ファイル、ユニットテスト、統合テスト、ライブラリ、およびワークスペース内の複数のパッケージの例をデバッグするための構成のセットアップ <!-- end of auto-generated comment: release notes by coderabbit.ai -->
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.
建物が一つの地物(Placemark)にまとまるように修正しました。
2024-02-14.15.12.37.mov
kmlデータ上では、
1gmlファイル...1Folder...nPlacemark...1MultiGeometry...nPolygon
の構成になっています。
Summary by CodeRabbit
multipolygon_to_kml関数の返り値をVec<KmlPolygon}に変更しました。indexed_multipolygon_to_kml関数の返り値をVec<KmlPolygon}に変更しました。multipolygon_to_kml_with_mapping関数の返り値をVec<KmlPolygon}に変更しました。polygon_to_kml_polygon_with_mappingという新しい関数を追加し、KmlPolygonを返します。polygon_to_kml_with_mapping関数の返り値をVec<KmlPolygon}に変更しました。polygon_to_kml関数の返り値をVec<KmlPolygon}に変更しました。indexed_polygon_to_kml関数の返り値をVec<KmlPolygon}に変更しました。