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

Jump to bottom

nusamai-geojson: 入力形式をTopLevelCityObjectに合わせる #62

Closed
sorami opened this issue Dec 13, 2023 · 0 comments · Fixed by #79
Closed

nusamai-geojson: 入力形式をTopLevelCityObjectに合わせる #62

sorami opened this issue Dec 13, 2023 · 0 comments · Fixed by #79
Assignees
@sorami
Labels
CityGML Source

Comments

@sorami
Copy link
Member

sorami commented Dec 13, 2023

@sorami sorami self-assigned this Dec 13, 2023
@sorami sorami mentioned this issue Dec 14, 2023
Closed
4 tasks
@sorami sorami mentioned this issue Dec 18, 2023
Merged
4 tasks
sorami added a commit that referenced this issue Dec 19, 2023
@sorami
nusamai-geosjon: トップレベル都市オブジェクトを変換（ジオメトリのみ） (#79)
cfecda8 
close #62

## 概要

- 従来のコードは、 `nusamai-geometry` の Geometryを直接扱って、変換していた
- → 「トップレベル都市オブジェクト」を入力として、GeoJSONへの変換処理を行う
  - #78 で用意されたものを利用している

`Geometries` にある `multipolygon`, `multilinestring`, `multipoint`
、それぞれをFeatureにしている。


## 変更

- [x] 変換処理の書き換え
  - `nusamai-geojson/src/conversion.rs`
  - `nusamai-geojson/src/lib.rs`
- [x] CityGMLファイルをパースしてGeoJSONを出力するexampleの追加
  - `nusamai-geojson/examples/gml2geojson.rs`
- その他
  - [x] Tauri側の暫定処理を、この変更に合わせて修正: `app/src-tauri/*`
- [x] いにしえのパースexampleの削除: `nusamai-geojson/examples/citygml_polygons.rs`

## 補足: 属性の扱い

属性はこのPRでは考慮していない。

トップレベル都市オブジェクトの ` cityobj: FeatureOrData<'a>` を扱う必要がある。

別途対応する #46
ciscorn pushed a commit that referenced this issue Dec 27, 2023
@sorami @ciscorn
nusamai-geosjon: トップレベル都市オブジェクトを変換（ジオメトリのみ） (#79)
1afbb99 
close #62

- 従来のコードは、 `nusamai-geometry` の Geometryを直接扱って、変換していた
- → 「トップレベル都市オブジェクト」を入力として、GeoJSONへの変換処理を行う
  - #78 で用意されたものを利用している

`Geometries` にある `multipolygon`, `multilinestring`, `multipoint`
、それぞれをFeatureにしている。

- [x] 変換処理の書き換え
  - `nusamai-geojson/src/conversion.rs`
  - `nusamai-geojson/src/lib.rs`
- [x] CityGMLファイルをパースしてGeoJSONを出力するexampleの追加
  - `nusamai-geojson/examples/gml2geojson.rs`
- その他
  - [x] Tauri側の暫定処理を、この変更に合わせて修正: `app/src-tauri/*`
- [x] いにしえのパースexampleの削除: `nusamai-geojson/examples/citygml_polygons.rs`

属性はこのPRでは考慮していない。

トップレベル都市オブジェクトの ` cityobj: FeatureOrData<'a>` を扱う必要がある。

別途対応する #46
ciscorn pushed a commit that referenced this issue Dec 27, 2023
@sorami @ciscorn
nusamai-geosjon: トップレベル都市オブジェクトを変換（ジオメトリのみ） (#79)
396f1a1 
close #62

- 従来のコードは、 `nusamai-geometry` の Geometryを直接扱って、変換していた
- → 「トップレベル都市オブジェクト」を入力として、GeoJSONへの変換処理を行う
  - #78 で用意されたものを利用している

`Geometries` にある `multipolygon`, `multilinestring`, `multipoint`
、それぞれをFeatureにしている。

- [x] 変換処理の書き換え
  - `nusamai-geojson/src/conversion.rs`
  - `nusamai-geojson/src/lib.rs`
- [x] CityGMLファイルをパースしてGeoJSONを出力するexampleの追加
  - `nusamai-geojson/examples/gml2geojson.rs`
- その他
  - [x] Tauri側の暫定処理を、この変更に合わせて修正: `app/src-tauri/*`
- [x] いにしえのパースexampleの削除: `nusamai-geojson/examples/citygml_polygons.rs`

属性はこのPRでは考慮していない。

トップレベル都市オブジェクトの ` cityobj: FeatureOrData<'a>` を扱う必要がある。

別途対応する #46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sorami sorami

Labels
CityGML Source
Projects
None yet
Milestone
No milestone
2 participants
@sorami @ciscorn