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

close #6: MADB の各話情報を episodes として追加 #10

Merged
merged 3 commits into from
Nov 10, 2016

Conversation

builtinnya
Copy link
Member

@builtinnya builtinnya commented Nov 9, 2016

ref: #6

※本 PR マージ後に #6 は閉じるため,他DB からの各話情報の追加については,新しい issue 及び PR を作成すること.

対応内容

  • MADB から得た各話情報を,対応するアニメに episodes として追加した.
    • started_at 及び ended_at については,MADB の「放送期間など」からパースして可能な限り時刻まで入れた.
  • README.mdepisodes に関する説明を追加した.

@builtinnya
Copy link
Member Author

@eru @honeshabri
🌟 クロスレビューよろしく 🌟

@eru
Copy link
Member

eru commented Nov 10, 2016

人間に出きるレビューの量じゃない、けどやる。

  • episodesの下の各キーの順番が、README.mdと違うのでちょっと理解しにくい(人間にはパースしにくい順番)。

  • README.mdに、episodesの記載順=放送順であることを明記したほうが良い。

  • 追加が多すぎて?意図しない差分(追加、削除)が発生している。

    • 削除されて、その下の方で追加みたいになってるので、本数は変わってない。
    -- madb_id: ANS000240200
    +- madb_id: ANS000238400
    medium: OVA
    duplicated:
    -  id: リカちゃんの日曜日
    -  title: リカちゃんの日曜日
    +  id: 原由子 眠れぬ夜の小さなお話
    +  title: 原由子 眠れぬ夜の小さなお話
    suffix:
    -  ruby: リカチャンノニチヨウビ
    +  ruby: ハラ ユウコ ネムレヌヨルノチイサナオハナシ

ここから下はMADBによる問題

  • titleで改行が含まれるものがある。

  • episodesが途中からしか入っていないアニメがある。

  • titleに本来はなかった日付が含まれているアニメが多数ある。

    • (11/27)のようにmm/ddが括弧書きで追加されている。
  • 鉄腕アトムの54話以降titleに話数が重複している。

    +  - ended_at: 1964/01/04 19:30
    +    title: 第54回 アルプスの決斗の巻*
    +    literal_title: 第54話 第54回 アルプスの決斗の巻*
    +    prefix: 第54話
    +    started_at: 1964/01/04 19:00
    +    ruby: ''
  • ドラえもんで、prefix = S1が複数ある。

    +  - ended_at: 1999/12/31 19:00
    +    title: S 未来を守れ! のび太VSアリ軍団
    +    literal_title: S1 S 未来を守れ! のび太VSアリ軍団
    +    prefix: S1
    +    started_at: 1999/12/31 18:50
    +    ruby: ''
    
    +  - ended_at: 2001/03/23 19:00
    +    title: 1617 タンポポ空を行く
    +    literal_title: S1 1617 タンポポ空を行く
    +    prefix: S1
    +    started_at: 2001/03/23 18:50
    +    ruby: ''
  • ドラえもんで、途中からtitleの数値の後ろの半角スペースが消える。

    • 1690 予言者ジャイアン
    • 1691メー演機
    • 1737物知~る
    • 1738 ナカナオリン <- ここで復活
    +  - ended_at: 2002/11/29
    +    title: 1690 予言者ジャイアン
    +    literal_title: 958 1690 予言者ジャイアン
    +    prefix: '958'
    +    started_at: 2002/11/29
    +    ruby: ''
    +  - ended_at: 2002/12/06
    +    title: 1691メー演機
    +    literal_title: 959 1691メー演機
    +    prefix: '959'
    +    started_at: 2002/12/06
    +    ruby: ''
    
    +  - ended_at: 2003/11/28
    +    title: 1737物知~る
    +    literal_title: 1001 1737物知~る
    +    prefix: '1001'
    +    started_at: 2003/11/28
    +    ruby: ''
    +  - ended_at: 2003/12/05
    +    title: 1738 ナカナオリン
    +    literal_title: 1002 1738 ナカナオリン
    +    prefix: '1002'
    +    started_at: 2003/12/05
    +    ruby: ''
  • 機動戦士SDガンダムprefixが全て-になっている。

    +  episodes:
    +  - ended_at: 1989/06/25
    +    title: 機動戦士SDガンダム MARK-Ⅱ 転がるコロニー事件
    +    literal_title: '- 機動戦士SDガンダム MARK-Ⅱ 転がるコロニー事件'
    +    prefix: '-'
    +    started_at: 1989/06/25
    +    ruby: ''
    +  - ended_at: 1989/06/25
    +    title: 機動戦士SDガンダム MARK-Ⅱ 元祖ガンダム迷場面集
    +    literal_title: '- 機動戦士SDガンダム MARK-Ⅱ 元祖ガンダム迷場面集'
    +    prefix: '-'
    +    started_at: 1989/06/25
    +    ruby: ''
  • それいけ!アンパンマンで、prefix = S1が複数ある。

    +  - ended_at: 1999/12/17
    +    title: アンパンマンと メリークリスマス! 前編
    +    literal_title: S1 アンパンマンと メリークリスマス! 前編
    +    prefix: S1
    +    started_at: 1999/12/17
    +    ruby: ''
    
    +  - ended_at: 2002/12/13
    +    title: 勇気のほのおとクリスマス(前編)
    +    literal_title: S1 勇気のほのおとクリスマス(前編)
    +    prefix: S1
    +    started_at: 2002/12/13
    +    ruby: ''
    
    +  - ended_at: 2006/12/22
    +    title: うたおう!おどろう!みんなのクリスマス
    +    literal_title: S1 うたおう!おどろう!みんなのクリスマス
    +    prefix: S1
    +    started_at: 2006/12/22
    +    ruby: ''
  • スパイラルゾーンprefixがいい感じじゃない。

    +  episodes:
    +  - ended_at: 1988/10/31
    +    title: Vol.1発進ゾーン・ライダー
    +    literal_title: 第1巻A Vol.1発進ゾーン・ライダー
    +    prefix: 第1巻A
    +    started_at: 1988/10/31
    +    ruby: ''
    +  - ended_at: 1988/10/31
    +    title: Vol.2悪魔の小箱
    +    literal_title: 第1巻B Vol.2悪魔の小箱
    +    prefix: 第1巻B
    +    started_at: 1988/10/31
    +    ruby: ''
  • ふしぎの海のナディアtitleにゴミ(「連続39回」)がついている。

  • ふしぎの海のナディアprefixがおかしい。

    +  episodes:
    +  - ended_at: 1990/04/13
    +    title: 第1回 エッフェル塔の少女 「連続39回」
    +    literal_title: 1 第1回 エッフェル塔の少女 「連続39回」
    +    prefix: '1'
    +    started_at: 1990/04/13
    +    ruby: ''
    +  - ended_at: 1990/04/20
    +    title: 第2回 小さな逃亡者 「連続39回」
    +    literal_title: 2 第2回 小さな逃亡者 「連続39回」
    +    prefix: '2'
    +    started_at: 1990/04/20
    +    ruby: ''
  • 新世紀エヴァンゲリオンprefixいい感じじゃない。

    +  episodes:
    +  - ended_at: 1995/10/04
    +    title: 第壱話 使徒、襲来
    +    literal_title: 第1話 第壱話 使徒、襲来
    +    prefix: 第1話
    +    started_at: 1995/10/04
    +    ruby: ''
  • 今、そこにいる僕 NOW AND THEN, HERE AND THEREで、prefixがいい感じじゃない。

    +  episodes:
    +  - ended_at: 1999/12/02
    +    title: 第八話 ひとりぼっちのふたり
    +    literal_title: 8 第八話 ひとりぼっちのふたり
    +    prefix: '8'
    +    started_at: 1999/12/02
    +    ruby: ''
  • **Mr.Digital TOKORO the comical cartoon [第2期]**で、prefixがいい感じじゃない。

    +  episodes:
    +  - ended_at: 2001/04/17
    +    title: '#126 サンダージョージ その①'
    +    literal_title: '126 #126 サンダージョージ その①'
    +    prefix: '126'
    +    started_at: 2001/04/17
    +    ruby: ''
  • フルーツバスケットで、prefixがいい感じじゃない。

    +  - ended_at: 2001/07/05
    +    title: 第一話・・・
    +    literal_title: 1 第一話・・・
    +    prefix: '1'
    +    started_at: 2001/07/05
    +    ruby: ''

[diff] Changes to 'animedb.yml' - line 114899 of 505535 22%

@builtinnya
Copy link
Member Author

builtinnya commented Nov 10, 2016

  • episodesの下の各キーの順番が、README.mdと違うのでちょっと理解しにくい(人間にはパースしにくい順番)。
  1. README.md と違うことが問題
  2. そもそも episodes 下のキーの順番が問題

1 は現状の episodes の順番に合わせる方向で解決可能.
2 は Dumper をカスタマイズすれば何とかなるかもしれんがだるい.

  • README.mdに、episodesの記載順=放送順であることを明記したほうが良い。

6fd21d9 で対応済み.

  • 追加が多すぎて?意図しない差分(追加、削除)が発生している。

diff のせい.

MADBの問題

prefix 問題はかなり多いので解決可能性が低そう.
title に「第N話」とか入ってたらそれを literal_title にして prefix を「第N話」にするとかはできそう.

@eru
Copy link
Member

eru commented Nov 10, 2016

  1. README.md と違うことが問題
  2. そもそも episodes 下のキーの順番が問題

1 は現状の episodes の順番に合わせる方向で解決可能.
2 は Dumper をカスタマイズすれば何とかなるかもしれんがだるい.

とりあえずはこのままにして、後追いでフォーマッター作って、precommitでそれと押して、全体の体裁を維持してくような運用でどうだろうか。
あくまでデータベースとして扱うのであれば、人の目で見れることはさほど重要でもないので。

@builtinnya
Copy link
Member Author

とりあえずはこのままにして、後追いでフォーマッター作って、precommitでそれと押して、全体の体裁を維持してくような運用でどうだろうか。
あくまでデータベースとして扱うのであれば、人の目で見れることはさほど重要でもないので。

それでいくか.
ただ,precommit はさすがに重いので,フォーマットに沿っていない場合を CI ではじき,formatter は animedb コマンドで実装する感じで.

Copy link
Member

@eru eru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@builtinnya
Copy link
Member Author

👍 👍 👍
🌟 🌟 🌟
👏 👏 👏

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.

None yet

3 participants