search

Tokyo-Metro-Gov / covid19

東京都 新型コロナウイルス感染症対策サイト / Tokyo COVID-19 Task Force website
https://stopcovid19.metro.tokyo.lg.jp/
MIT License
6.26k stars 1.97k forks source link

新規グラフ追加：新宿区への来訪者数の推移 #1318

Closed ocean3fish closed 4 years ago

ocean3fish commented 4 years ago

新規グラフの作成にご協力いただける方を募集しています。 ご不明な点は随時お答えします。

利用データ

見せ方

注記事項

完成イメージ

shgtkshruch commented 4 years ago

まだどなたもアサインされていなければ、実装させていただきたいです。

ocean3fish commented 4 years ago

@shgtkshruch ありがとうございます！ ぜひよろしくお願いします！

shgtkshruch commented 4 years ago

頂いたデータとスクリーンショットを元に実装をすすめました！

認識の齟齬などがありましたら、プルリクエストでコメントをいただければと思います。

また、グラフタイトルや注釈、ラベル表記等が確定しましたらこちらの issue でお知らせいただけると嬉しいです。 文言が確定しましたら、プルリクエストに反映させていただきます。

ocean3fish commented 4 years ago

@shgtkshruch 早速ありがとうございます！大変正確に仕上げて頂き感謝申し上げます！ 3点ほど調整・変更をお願いしたい事項があるのですが、ご対応頂くことは可能でしょうか？

shgtkshruch commented 4 years ago

@ocean3fish 確認いただきまして、ありがとうございます！ コメントいただいた3点ともに修正可能ですので、これから対応いたします :D （グラフ図の更新ありがとうございます、わかりやすくて助かりました 🙏 ）

shgtkshruch commented 4 years ago

@ocean3fish 数値処理やグラフの対象データを修正いたしました！

1335

実数値

スクリーンショット 2020-03-14 19 09 36

ツールチップ

スクリーンショット 2020-03-14 19 09 50

相対値

スクリーンショット 2020-03-14 19 09 41

ocean3fish commented 4 years ago

@shgtkshruch ご対応ありがとうございます！ プレビュー拝見しまして、依頼させて頂いた3点の変更について問題ないことを確認しました！

追加で1点だけ申し訳ありません、実数値のグラフの方、 軸に0を含めない実装にご変更頂いているのですが、 数値の大小を比較する目的の棒グラフでは0をカットしない方が良いとされています。 お手数ですが、0を含める形に戻して頂けるでしょうか🙏

shgtkshruch commented 4 years ago

@ocean3fish

ご確認いただきまして、ありがとうございます！ Y軸、確かにスクリーンショットと違いましたね... こちらのコミットで 0 を含めるように修正いたしました :) https://github.com/tokyo-metropolitan-gov/covid19/pull/1335/commits/89ddd12fd41bf67c24c76252c7aca26ca2fca5d4

スクリーンショット 2020-03-14 22 12 18

ocean3fish commented 4 years ago

@shgtkshruch 早速ありがとうございました！ 丁寧なご対応感謝いたします！

以降のグラフタイトルや注釈等は週明けに確定する見込みのため、またこちらでお知らせさせてください。 よろしくお願いします。

shgtkshruch commented 4 years ago

@ocean3fish

以降のグラフタイトルや注釈等は週明けに確定する見込みのため、またこちらでお知らせさせてください。

承知しました！ 文言等の調整ありがとうございます。 引き続き、よろしくお願いします！

ocean3fish commented 4 years ago

@shgtkshruch 表示するテキストの内容が確定したのと、表示するデータの変更の要望があったのですが、ご対応頂くことは可能でしょうか？

変更点

  1. 実数値のグラフは削除（実装頂いたところ大変申し訳ありません…！）
  2. 集計対象データを平日のみにする →よろしければ2-3月の日付を平日と休祝日に分類したデータを作成してあるのでご利用ください：calendar.csv
  3. 棒グラフは2/10の週以降を表示
  4. タイトル変更： 新宿区エリアの来訪者推移（参考値）
  5. 注釈変更： 
    2月3日～2月7日の来訪者数 (※1) の平均値 (※2) を 基準としたときの相対値
※1) ヤフーに蓄積された位置情報データなどを元に算出した参考値
※2) 土・日・祝日を除く7:30~8:30の1週間平均値
  6. データの掲載期間を明示（グラフと出典表示の間に追加） ※本データは2020年3月31日までの掲載となります
  7. コーピーライト表記変更 
    (C)Yahoo Japan
出典：[ヤフー・データソリューション](https://ds.yahoo.co.jp/datapolicy/)

    実装イメージ

    スクリーンショット 2020-03-17 20 49 20
shgtkshruch commented 4 years ago

@ocean3fish

文言等の確認ありがとうございます！

  1. 実数値のグラフは削除

承知しました！ 実装シンプルになるので、大丈夫です :)

  1. 集計対象データを平日のみにする

こちらも対応可能です。 平日と休祝日に分類したデータもありがとうございます 🙏

内部的に採用している日付関連のライブラリ（day.js）でも平日と休日の判定ができるのですが、こちらを使うと、 - 最初に頂いたデータ構造がそのまま使える（平日・休日のフラグがなくても OK） - もし休日のデータがあってもプログラム的に入らないことを保証できる

ので、今回はこちらで実装しようと思っています。

[追記] すみません、土日は判定できましたが祝日は取れなかったので、用意いただいた CSV をもとにデータ構造を変えようと思いますmm [/追記]

また、3 ~ 7 で頂いた表示期間や文言の修正も対応可能です。 文言が少し増えるので、他のグラフと並べたときに余白などの調整が必要かもしれませんが、このあたりはやりながら調整させていただきます。

頂いたイメージを元に実装をすすめて、明日にはこちらの issue で共有させていただきます。

shgtkshruch commented 4 years ago

@ocean3fish

昨日いただいた変更点を実装いたしました！ お手すきの際にご確認お願いいたします。

Netlify のプレビューのリンクはこちらになります。 https://deploy-preview-1335--dev-covid19-tokyo.netlify.com/

スクリーンショット 2020-03-18 19 20 51

ocean3fish commented 4 years ago

@shgtkshruch 迅速に仕上げて頂き本当にありがとうございます！ 内部で本番用データの準備に時間を要しているようで、公開までにもう少しお待ち頂く形になってしまい申し訳ありません🙏 PRをオープンにして頂けるタイミングになりましたらまたご連絡させて頂きます。

MaySoMusician commented 4 years ago

1852 で要望があがった翻訳キーを #1857 にて追加しました

"新宿区エリアの来訪者推移（参考値）": "新宿区エリアの来訪者推移（参考値）",
"2月3日～2月7日の来訪者数 (※1) の平均値 (※2) を 基準としたときの相対値": "2月3日～2月7日の来訪者数 (※1) の平均値 (※2) を 基準としたときの相対値",
"※1) ヤフーに蓄積された位置情報データなどを元に算出した参考値": "※1) ヤフーに蓄積された位置情報データなどを元に算出した参考値",
"※2) 土・日・祝日を除く7:30~8:30の1週間平均値": "※2) 土・日・祝日を除く7:30~8:30の1週間平均値",
"※本データは2020年3月31日までの掲載となります": "※本データは2020年3月31日までの掲載となります",
"出典: {source}": "出典: {source}",
"ヤフー・データソリューション": "ヤフー・データソリューション"

「出典: ヤフー・データソリューション」については <i18n> タグによる入れ子構造が使用できます

期間：M/D〜M/D については 期間: {duration} が既に存在しますのでこちらが使えるはずです