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

"specifications.md" における "LayerGroup" の "id" は必須ではなく仕様または記述の何れかが誤りの可能性が高い #2

Open
usagi opened this issue Sep 9, 2020 · 1 comment

Comments

@usagi
Copy link

usagi commented Sep 9, 2020

対象の位置

"LayerGroup" 項"属性名" idデフォルト 列 の (必須) の記述

不具合と推量する内容

  • 現在の記述: (必須)
  • 正しいかもしれない記述: "" またはそもそも LayerGroup に id は仕様上定義されない

image

推量に至る経緯

gsimaps でのすべての Layers.txt の実装における LayerGroupid について確認してはいませんが、 Layers1.txt 及び Layers.txttoptrue に設定されている layers_topic_20200703oame.txt のように主要または目立つ実装が仕様と異なる事から、仕様または仕様の記述または実装の何れかに不具合があると推量します。

期待される対応

  1. "specifications.md" の記述上の誤りで、ほんらい仕様では LayerGroupid は定義されない場合 => 記述を修正した上で告知を出して周知する
  2. 仕様の不具合で LayerGroupid は定義されないはずだったが定義される事になってしまっていた場合 => 仕様を修正し告知
  3. 仕様としては LayerGroupid は必須で gsimaps での実装に不具合がある場合 => gsimaps へ修正 Issue を立て、この Issue へリンクし、この Issue は Close して対応を gsimaps 側へ引き継ぐ
  4. 仕様としては LayerGroupid は必須だが、主要な実装で未定義で使用されている状況に対応し仕様側を修正 => 仕様を更新し LayerGroupid(必須) から "" または null, undefined 等の無効値を デフォルト の仕様に修正し、告知
@johofukyu
Copy link
Member

ご指摘ありがとうございます。

LayerGroupのid属性は、例えばlayers7.txtの「5mメッシュDEM(航空レーザ測量)の提供地域と作業年度」のように、記述することで配下のLayerをまとめて1つのレイヤのように表示させることができます(→地理院地図で表示した例)。

本レポジトリのissue #4と併せて修正いたします。
具体的には、LayerGroupのid属性については、「省略可否」の列を「可 ※」とし、表の外(下部)に「※ LayerGroup配下のLayerをまとめて1つのレイヤのように表示させたい場合、id属性を記述する。」と記載することといたします。

よろしくお願いいたします。

johofukyu added a commit that referenced this issue Oct 9, 2020
issue #2, #4 を反映。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants