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

顔チャート 作成 #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

kazun2112
Copy link

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

@hipsrinoky
Copy link
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