eukarya-inc / plateauutils Goto Github PK

Running CityGMLParser.parse_and_download() takes so long time. Is it possible to show a progress bar while parsing CityGML?

My suggestion is adding progressbar at the line 180 of city_gml_parser.py (there might be better location to add progress bar).

#city_gmle_parser.py
from progressbar import progressbar  # anywhere before

        for city_object_member in progressbar(city_object_members):  # line 180

Regards,
Akito

• Set None value by no values

CityGMLParser の parse にて、shutil.unpack_archive せずに with zipfile.ZipFile 内でパースすることは可能でしょうか？

参考資料 https://hacker.trickstorm.com/?p=275

Colabで実行している時に、Google Driveをマウントするとそちらの方に解凍ファイルが流れ込み容量圧迫してしまいます。

README.rstに最低限の開発のことしか書いてないので、doc/以下含めてドキュメントを充実させる。

作業内容

• Plateau Utilsのパースする属性に下記の内容を追加する
  bldg:usage,real_estate_id_of_building,real_estate_id_of_building_unit_ownership

• OSSとして公開後、第三者が任意の属性をパース対象として追加したい場合、スクリプトにどのような変更を加えればよいかをドキュメント化（日本語）する。

期限

3月中旬 *要相談

#1 の実装のCUIインターフェイスを作成する。
これはデバッグ用に使われる。

• supports classification output using specify py3dtiles
• shrink 3DTiles output (remove dem is nan)
• png output supports lower zooms(8-14)

スキーマがv2のみ対応しているので、v1及びv3のサポートを行う。

Polygonを入力し、該当する地域メッシュコードの配列を返すクラスを作成する。
タイルの配列を返すクラスを作成するため、基底クラスも作成する。

#3 で作成した抽象クラスを用いて、Polygonを入力してタイルのリストを返すクラスを作成する。

以下の機能を持つベースとなるクラスを作成する。

• Plateauのファイルをダウンロード
• ファイルの解析機能のベース

このクラスから以下の機能を実装する予定

• Buildingのデータ解析
• MVTファイルのリスト出力

https://gitlab.com/py3dtiles/py3dtiles/-/issues/194

With parsing Kurume's CityGML, I found there are some regions (or tiles) where parsing fails.

The boundary I use for running CityGMLParser and CityGMLParser.download_and_parse is the following.

    [
      [130.41249721501615, 33.224722548534864],
      [130.41249721501615, 33.348],
      [130.59, 33.348],
      [130.59, 33.224722548534864],
      [130.41249721501615, 33.224722548534864]
    ]

I made a scatter plot of building locations (yellow dots) over DEM. There is a gap or parsing failures around the latitude of 130.500 degree.

pyproject.toml references README.rst and when you make new package, it needs some tuning for pyproject.toml, because ReST document is default format for Python.
I recommend to revert README.rst from README.md.

現状APIドキュメントに統一感がないのでreSTで書き直す。

コマンドラインからPointデータを受け取り、地域メッシュコードを返すCUIインターフェイスを用意する。
この関数はコマンドラインから確認用のため、本体自体はすでに実装済みである。

share vscode setting for black

gh-pages branch use docfx's document now.

• use secret API
• use public API

開発の補助のため、建物群のサンプルデータを作成する

"building_structure_type" の値がこのままだと理解できません。何らかの変換をかませて「木造」とか「鉄筋コンクリート」の様な値にすることは可能でしょうか？

以下のコードを２度実行した場合に FileExistsError が上がりました。

from plateauutils.parser.city_gml_parser import CityGMLParser
parser = CityGMLParser(from_wkt(building_bound))
buildings = parser.download_and_parse(url_CityGML, "/tmp")

raise元は abc/plateau_parser.py の 34行目~36行目、 _download 内。

        # 保存先パスが既に存在すればエラー
        if exists(saved_path):
            raise FileExistsError(f"{saved_path} already exists.")

CityGMLParser.download_and_parse が

        saved_path = self._download(url, target_dir)
        return self.parse(saved_path)

なので、 FileExistsError を raiseせずに、saved_path を返すのではダメなのでしょうか？

地域メッシュコードからPolygon(BBOX)を返す関数を作成する。
この関数は指定したPolygonが地域メッシュコードに含まれるかどうかを判定するのに利用する。

code coverage を導入してテストの網羅率を計算する。

setupを書き終えてPyPIに公開する。