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.

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

nusamai-gpkg: Sinkの追加 #121

Closed
sorami opened this issue Dec 26, 2023 · 0 comments · Fixed by #136
Closed

nusamai-gpkg: Sinkの追加 #121

sorami opened this issue Dec 26, 2023 · 0 comments · Fixed by #136
Assignees
Labels
GeoPackage Output Fomat Output format drivers

Comments

@sorami
Copy link
Member

sorami commented Dec 26, 2023

#89 でのミニマムな実装をもとに、GeoPackage用のSinkを用意する。

参考: GeoJSONでの対応例 - #87

これにより、CLIで、CityGMLファイルをGeoPackageへ変換して出力することも可能になる。↓のようなイメージ

$ time cargo run --release plateau/22203_numazu-shi_2021_citygml_4_op/udx/bldg/*.gml --sink gpkg
@sorami sorami added the Output Fomat Output format drivers label Dec 26, 2023
sorami added a commit that referenced this issue Dec 27, 2023
#89 を踏まえて、トップレベル都市オブジェクトをGeoPackageへ追加する部分を実装。

参考(Notion): [GeoPackage
メモ](https://www.notion.so/GeoPackage-56b53cc7ee5f4270a7df151d84bfb290)
(テーブルやバイナリの構造など)

## 大まかな流れ

- `GpkgHandler::init` (`src/handler.rs`)
  - `sql/init.sql` で、GeoPackage用のテーブルを作成
  - `sql/mpoly3d.sql` で、3Dマルチポリゴン用のテーブルを用意
- `GpkgHandler::add_object`
  - トップレベル都市オブジェクトを受け取り、DBへ追加
  - ジオメトリのバイナリを作る部分は `src/geometry.rs` にある

exampleの実行例:

```
$  cargo run --example gml2gpkg ~/plateau/13100_tokyo23-ku_2022_citygml_1_2_op/udx/bldg/53392546_bldg_6697_2_op.gml
```

## このPRで対応しない想定のイシュー

- #119
- #120
- #121
- #122
- #124 
- #125
@sorami sorami self-assigned this Dec 28, 2023
ciscorn added a commit that referenced this issue Jan 5, 2024
close #121

#89 , #129 での、Gpkg変換の初期実装をもとに、全体の流れに組み込む(sinkを作る)。

とりあえず書いてみたコード例のようなものなので、コメントいただきたいです。

## 悩み

SQLx(async)の取り扱い。とりあえず、Sinkの中でブロックしてやってみているが、こう言う方法でいいのか?

---------

Co-authored-by: Taku Fukada <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GeoPackage Output Fomat Output format drivers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants