Skip to content

顔チャート 作成#30

Open
kazun2112 wants to merge 1 commit intoe2d3:masterfrom
kazun2112:master
Open

顔チャート 作成#30
kazun2112 wants to merge 1 commit intoe2d3:masterfrom
kazun2112:master

Conversation

@kazun2112
Copy link
Copy Markdown

顔チャートを作成しました。マージしてください。

@hipsrinoky
Copy link
Copy Markdown
Contributor

お待たせしました。チェックしています。若干問題がありましたので、下記に記します。

【顔画像について】
/contribからのパスを利用しているようですが、環境に依存しますので別の方法でお願いします。
例えば、jpglink項目には、ファイル名(red.jpgなど)だけ記載し、JavaScript側で、

baseUrl + '/face/' + d[jpglink]

などとする方が望ましくなります。

もしくは、「jpglink」ではなく、「color」のようにしておき、

baseUrl + '/face/' + d[color] + '.jpg'

として、データには「red」 「blue」と色名を入れるほうが、ユーザーは理解しやすいかもしれません。

【csvの項目について】
現状、ヘッダーはdate,ParamX,ParamY,ParamZ,jpglinkのようになっていますが、
dateはnameの間違いでしょうか?

【データ更新時の挙動について】
データ更新時に、オブジェクトの位置が変わるのではなく、オブジェクトが追加されてしまいます。

  • オブジェクトが移動する
  • 一度全てのオブジェクトを消して再描画

などの対策をお願いします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants