search

ViBii / mitemiru

2 stars 0 forks source link

[IR] US04-F01-SP01: グラフ仕様の設計 8.0h #312

Closed yaginuuu closed 8 years ago

yaginuuu commented 8 years ago

関連事前調査： #261

グラフの概要

Altairzym commented 8 years ago

@sjc1209 @ryoki1118 レビューお願いします。

okjsc1118 commented 8 years ago

@Altairzym これってコミット数のグラフとは別で作る事を想定してる？

Altairzym commented 8 years ago

@ryoki1118 そうですね。。。

sjc1209 commented 8 years ago

わかりやすいと思う。いいと思う。ちなみに、＠の人は間違っている。

Altairzym commented 8 years ago

@sjc1209 え？その人は？誰？

Altairzym commented 8 years ago

@yaginuuu @ryoki1118 @sjc1209 昨日のミーティングの内容をまとめて、この様な感じに修正しました。どうでしょうか？意見お願いします。

sjc1209 commented 8 years ago

レビュー専用のアカウントです。

okjsc1118 commented 8 years ago

@Altairzym グラフ形式の前に、データの集計方法の話も書いてもらえる？ そこが抜けてるとこのグラフが適切か判断できないかな...

Altairzym commented 8 years ago

@ryoki1118 なるほど

Altairzym commented 8 years ago

@ryoki1118 @sjc1209 データ取得部分を考えた、どうでしょうか？

okjsc1118 commented 8 years ago

@Altairzym データの技術的な収集方法の話じゃなくて、グラフとして表現する数値の集計方法を書いて欲しかった...。データ収集の流れは何となく書いてあるんだけど、結局どんなデータが生成されて、どんな数字として表現されるのかが全くわからないんだよね。

・1つのファイルに10回コミットした場合は、1回としてカウントするのか、それとも10回としてカウントするのか ・開発者ごとに編集ファイルの拡張子をパーセンテージで表現するけど、具体的にはどういう計算で導き出すのか

とりあえずこの辺りがわからないから教えて欲しいかな。

Altairzym commented 8 years ago

@ryoki1118 そうだね、確かに。。。修正します

Altairzym commented 8 years ago

@ryoki1118 @sjc1209 またよろしくね

okjsc1118 commented 8 years ago

@Altairzym

(注意：ここの回数はコミットした回数である。10回コミットすると10回と計算する)

1回のコミットでfoo.jsとbar.jsを編集した場合は、jsの編集回数はいくつになるの？

Altairzym commented 8 years ago

@ryoki1118 実際の例を言うと、以下のURLの集計結果としてはhttps://api.github.com/repos/ViBii/mitemiru/commits/aae4413aa10520167a741d38c5ef3c0d15557ae5 js:2回 rb:1回 html.erb:1回

okjsc1118 commented 8 years ago

@Altairzym

@ryoki1118 実際の例を言うと、以下のURLの集計結果としてはhttps://api.github.com/repos/ViBii/mitemiru/commits/aae4413aa10520167a741d38c5ef3c0d15557ae5 js:2回 rb:1回 html.erb:1回

この場合、「全てのファイルを編集した総回数」は4回？

okjsc1118 commented 8 years ago

@Altairzym あと別件だけど、他人のブランチをマージした場合ってどうやってカウントするの？ 例えばこれとか(孫くんのケース) https://api.github.com/repos/ViBii/mitemiru/commits?author=yoshisjc113@gmail.com https://api.github.com/repos/ViBii/mitemiru/commits/ee71d38095407ef708ef1c960c92c0ea774587f9

okjsc1118 commented 8 years ago

@Altairzym

261 にも書いてあるけど、Mergeするとparentsが2つになるから、そこを見て取り除けば適切な回数をカウントできるかも。

2015-12-02 19 20 41
Altairzym commented 8 years ago

@ryoki1118 はい、ありがとう！その部分を参考しながらやります！

okjsc1118 commented 8 years ago

@Altairzym テスト用のブランチで検証した結果

  1. Masterにブランチがマージされるまで、コミットログは取得できない
  2. マージすると、今までコミットした人のコミットログに反映される(今回の場合だと4回が大木のコミットとしてログに残る) https://api.github.com/repos/ViBii/mitemiru/commits?author=r1118oboe@gmail.com 2015-12-02 19 37 42
  3. 2に加えて、Merge pull requestというコミットログが マージした人のコミットログ として反映される(今回の場合だと、一民のコミットとしてログが残る) https://api.github.com/repos/ViBii/mitemiru/commits?author=zouyimin88@gmail.com 2015-12-02 19 37 37

今回のケースだと、大木のコミット4回と一民のコミット1回の合計5回がカウントされるようです。

さらに、一民のマージコミットの履歴の詳細を見ると、これまでに俺がコミットした内容が全て含まれています(実際には一民は一度も編集していない) https://api.github.com/repos/ViBii/mitemiru/commits/16790ad5e883439dd82fb37380e477196d4b74fe 2015-12-02 19 45 35

以上の事を踏まえると、マージした時のコミットログを除いた他のコミットログを全て集計すれば、正確な情報を取り出せると思います。

okjsc1118 commented 8 years ago

@Altairzym ちなみに前述のとおり、マージしたコミットはparents(親ブランチ)が2つになっているので、それを除けばなんとかなると思う

Altairzym commented 8 years ago

@ryoki1118 了解です、今この二つのparentsを除くロジックを考えている、ありがとう

Altairzym commented 8 years ago

@ryoki1118 記述部分もう修正した、レビューお願い

okjsc1118 commented 8 years ago

@Altairzym InProgressだけどレビューしていいの？ちなみにどこが変わったの？

Altairzym commented 8 years ago

@ryoki1118 「IR」ですよ、レビューお願い！！！！

okjsc1118 commented 8 years ago

@Altairzym これ実装じゃないでしょ笑 Doneに置いてくださいー

Altairzym commented 8 years ago

@ryoki1118 修正箇所は image

okjsc1118 commented 8 years ago

@Altairzym だいたいおっけーかな ってもグラフってトラッカーのやつと全く同じってことだよね？笑 何だか変な感じがするんだよなぁ...

Altairzym commented 8 years ago

@ryoki1118 そのグラフはエクセルで作成したが、確かに既存のやつと似ている

okjsc1118 commented 8 years ago

@Altairzym まぁええか。

okjsc1118 commented 8 years ago

@yaginuuu @sjc1209 レビューよろしくね

yaginuuu commented 8 years ago

おｋです.

Altairzym commented 8 years ago

closeします