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

read_fgr_dem.Rの修正 #3

Merged
merged 3 commits into from
Dec 18, 2018
Merged

read_fgr_dem.Rの修正 #3

merged 3 commits into from
Dec 18, 2018

Conversation

nonsabotage
Copy link
Contributor

FG-GML-5638-04-DEM5Aで試したところ,不具合が生じたため3点修正しています.

@nonsabotage nonsabotage changed the title read_fgr_dem.Rno read_fgr_dem.Rの修正 Dec 17, 2018
@uribo
Copy link
Owner

uribo commented Dec 17, 2018

ご報告ありがとうございます。
また不備については申し訳ございませんでした。

FG-GML-5638-04-DEM5A のファイルということですが、具体的にはどのXMLファイル(3次メッシュ)が問題になりますでしょうか。

こちらでの確認のためにお教えいただけますでしょうか。

@uribo uribo added the bug Something isn't working label Dec 17, 2018
@nonsabotage
Copy link
Contributor Author

たとえば,FG-GML-5638-04-20-DEM5A-20151125.xmlです.

@uribo
Copy link
Owner

uribo commented Dec 17, 2018

ありがとうございます。
該当のファイルは仕様書4.0のものではないでしょうか(最新は4.1)。

ドキュメントを整備していないのがいけないのですが、現在対応しているのは4.1になります。
現在ダウンロード可能な4.1のファイルでは下記のようになります。

library(fgdr)
read_fgd_dem("FG-GML-5638-04-20-DEM5A-20161001.xml", resolution = 5)
#>  # A tibble: 33,750 x 2
#>  type       value
#>  <chr>      <dbl>
#>    1 データなし -9999
#>  2 データなし -9999
#>  3 データなし -9999
#>  4 データなし -9999
#>  5 データなし -9999
#>  6 データなし -9999
#>  7 データなし -9999
#>  8 データなし -9999
#>  9 データなし -9999
#>  10 データなし -9999
#>  # ... with 33,740 more rows

@uribo
Copy link
Owner

uribo commented Dec 18, 2018

バージョンの記載については #4 で対応させていただきます。

@nonsabotage
Copy link
Contributor Author

nonsabotage commented Dec 18, 2018

該当ファイルはv4であり,最新はv4.1でした.指定したファイルが古いもので申し訳ありません.一方でv4.1で変わったのはエンコーディングだけと認識しており,実際,新しいファイルのもとでも同様の不具合が生じました.

FG-GML-5638-04-20-DEM5A-20161001.xmlの読み込み結果をプロットすると,期待しているのは右のグラフでありますが,左のグラフのように右にずれている感じになります.uribo様の環境では左のグラフのようにはプロットはされないのでしょうか?

a

※ プロットする際に-9999をNAにしています

@uribo
Copy link
Owner

uribo commented Dec 18, 2018

確認しました。確かに左図のようになってしまいますね。
いただいたPull requestを反映すると欠損の処理が正しくできているように思えます。

他にコミットなければマージさせていただければと思います!

@nonsabotage
Copy link
Contributor Author

githubに不慣れなためご迷惑をおかけしたと思います.申し訳ありません.
他のコミットはありません.よろしくお願いいたします.

@uribo uribo merged commit 2f4707d into uribo:master Dec 18, 2018
@uribo
Copy link
Owner

uribo commented Dec 18, 2018

いえいえ!明快な説明、ありがとうございました。

535e061 でコントリビューションの内容を書かせいただきました。また何かありましたらよろしくお願いいたします。

@nonsabotage nonsabotage deleted the dev branch December 19, 2018 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants