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

ルビの修正 / id, title の整理(#20対応) #19

Merged
merged 15 commits into from
Nov 14, 2016

Conversation

haruyama
Copy link
Contributor

  • test_ruby() を復活させました
    • エラー時に madb_id がある場合は 表示するようにしました
  • docker-compose.yml に test service を追加しました
  • テストが通るように ruby を直しました
    • ちょっと調べただけでは読み方がわからないものも多かったので 適当に入れたものもあります

見直しから WIP を取ります

@@ -649646,7 +649646,7 @@
id: ルパン三世 VS 名探偵コナン THE MOVIE
title: ルパン三世 VS 名探偵コナン THE MOVIE
suffix:
ruby: ルパン3セイ タイ メイタンテイコナン ザムービー
ruby: ルパンサンセイ タイ メイタンテイコナン ザムービー
Copy link
Member

Choose a reason for hiding this comment

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

✕「タイ」
◯「バーサス」
ガキ使限定CMの音声で確認

@@ -51056,7 +51056,7 @@
id: みんなのうた「ショボクジラ・チビコブラ」
title: みんなのうた「ショボクジラ・チビコブラ」
suffix:
ruby: ミンナノウタショボクジラ・チビコブラ
ruby: ミンナノウタ ショボクジラ・チビコブラ
Copy link
Member

Choose a reason for hiding this comment

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

"・"が残っている

@@ -18169,7 +18169,7 @@
id: みんなのうた「牧場の朝」
title: みんなのうた「牧場の朝」
suffix:
ruby: ミンナノウタボクジョウノアサ
ruby: ミンナノウタ ボクジョウノアサ
Copy link
Member

Choose a reason for hiding this comment

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

✕「ボクジョウ」
◯「マキバ」

@@ -16809,7 +16809,7 @@
id: みんなのうた「ジロマジトンド・ジロトンド」
title: みんなのうた「ジロマジトンド・ジロトンド」
suffix:
ruby: ミンナノウタジロマジトンド・ジロトンド
ruby: ミンナノウタ ジロマジトンド・ジロトンド
Copy link
Member

Choose a reason for hiding this comment

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

"・"が残っている

@@ -15829,7 +15829,7 @@
id: みんなのうた「トム・ビリビ」
title: みんなのうた「トム・ビリビ」
suffix:
ruby: ミンナノウタトム・ビリビ
ruby: ミンナノウタ トム・ビリビ
Copy link
Member

Choose a reason for hiding this comment

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

"・"が残っている

@@ -14809,7 +14809,7 @@
id: エイトマン ロボット007 光線銃レーザー(ダブり)
title: エイトマン ロボット007 光線銃レーザー
suffix: ダブり
ruby: エイトマン ロボット007 コウセンジュウレーザー
ruby: エイトマン ロボットゼロゼロセブン コウセンジュウレーザー
Copy link
Member

Choose a reason for hiding this comment

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

✕「ゼロゼロセブン」
◯「ゼロゼロナナ」
そういうキャラ名

@@ -14829,7 +14829,7 @@
id: エイトマン ロボット007 光線銃レーザー
title: エイトマン ロボット007 光線銃レーザー
suffix:
ruby: エイトマン ロボット007 コウセンジュウレーザー
ruby: エイトマン ロボットゼロゼロセブン コウセンジュウレーザー
Copy link
Member

Choose a reason for hiding this comment

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

✕「ゼロゼロセブン」
◯「ゼロゼロナナ」
こっちも

@@ -13024,7 +13024,7 @@
id: みんなのうた「サンタはどこにいる?」
title: みんなのうた「サンタはどこにいる?」
suffix:
ruby: ミンナノウタサンタハドコニイル?
ruby: ミンナノウタ サンタハドコニイル?
Copy link
Member

Choose a reason for hiding this comment

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

"?"が残っている

@haruyama
Copy link
Contributor Author

commit 漏れがありました.

バーサスとヴァーサスが混在してるので 統一してしまってもいいかもしれません

@honeshabri
Copy link
Member

バーサスとヴァーサスが混在してるので 統一してしまってもいいかもしれません

『A-Ko The VS』は「ヴァーサス」と読むことになっていますので、下手に統一しないほうがいいです

@haruyama
Copy link
Contributor Author

VS 今回の変更範囲では「ヴァーサス」としていますが「バーサス」のほうがよいでしょうか

@haruyama
Copy link
Contributor Author

気になった項目

  • ANS000452200
    • id: ギャラクシーエンジェル(第3期) / title: ギャラクシーエンジェル / ruby: ギャラクシーエンジェルエース / ギャラクシーエンジェルダブルエース(変更前)
    • ruby は ギャラクシーエンジェルダブルエース としました. id や title も ギャラクシーエンジェルAA にするのが適当かもしれません
  • ANS000156600
  • ANS000806400
    • id: みなしごハッチ ママにだかれて / title: みなしごハッチ ママにだかれて / ruby: コンチュウモノガタリ ミナシゴハッチ ママニダカレテ
    • id, title, ruby で統一すべき?
  • ANS000604400
    • id: それいけ!アンパンマン おともだちシリーズ/ファンタジー / title: それいけ!アンパンマン おともだちシリーズ/ファンタジー/ ruby: ソレイケアンパンマン オトモダチシリーズ ファンタジー バタコサントソラトブマフラー
    • おいでよ!アンパンマン沼 バタコさんと空飛ぶマフラー アンパンまんの中の一話のようです. ruby にだけ バタコサントソラトブマフラー の情報があるので id, title に反映させたほうがいいかもしれません
  • ANS008051700
    • id: ガリレイドンナ / title: ガリレイドンナ / ruby: ガリレイドンナ Storia di tre sorelle a caccia di un mistero (変更前)
    • Storia 以下はタイトルに含めなくてよいのではないかと思い ruby も ガリレイドンナ のみにしています

@honeshabri
Copy link
Member

VS 今回の変更範囲では「ヴァーサス」としていますが「バーサス」のほうがよいでしょうか

公式の資料(Webサイト、PVなど)で、特に読み方に触れてない限りは、「バーサス」に統一してください
Googleで検索それぞれ検索すると

  • バーサス: 約 3,550,000 件
  • ヴァーサス: 約 1,760,000 件

のように「バーサス」の方が一般的なので

@builtinnya
Copy link
Member

@haruyama
ありがとうございます.
「気になった項目」については,本 PR で解決だと重いのであとで #20 から小分けするなりして対応します.

@honeshabri
Copy link
Member

honeshabri commented Nov 14, 2016

ギャラクシーエンジェル(第3期)

Wikipediaによると

映像作品タイトルは1 - 26話がギャラクシーエンジェルA(エース)、27 - 51話がギャラクシーエンジェルAA(ダブルエース)、54,55話がギャラクシーエンジェルS(エス)。

どれか特定の名前をつけるよりも第3期でくくってしまった方がいいです。

なおANS000516000の場合は第8話まで『流星戦隊ムスメット』で第9話 - 第11話の間『流星戦隊オトメット』となり、第12話で『帰ってきた 流星戦隊ムスメット』となりましたが、基本は『流星戦隊ムスメット』なので『流星戦隊ムスメット』に統一します。

@haruyama
Copy link
Contributor Author

このPR内の変更について VS の ruby として 「バーサス」 を用いるようにしました.
ただし,

- madb_id: ANS000640600
  id: ルパン三世VS名探偵コナン
  title: ルパン三世VS名探偵コナン
  ruby(変更前): ルパンサンセイ ヴァーサス メイタンテイコナン / ルパンサンセイ ブイエス メイタンテイコナン / ルパンサンセイ タイ メイタンテイコナン

については 元の ruby に ヴァーサスがあるので ruby: ルパンサンセイ ヴァーサス メイタンテイコナン としています. ここもバーサスにしたほうがよければそのようにします

@haruyama
Copy link
Contributor Author

ギャラクシーエンジェル(第3期)

ruby としては ギャラクシーエンジェル でしょうか ギャラクシーエンジェル ダイサンキ でしょうか?

ANS000516000 については 流星戦隊ムスメット の id, title, ruby だけ残せばいいでしょうか?

@honeshabri
Copy link
Member

ギャラクシーエンジェル(第3期)

rubyはギャラクシーエンジェルでお願いします
rubyはtitleの読み仮名を書く方針です
idはtitle + suffixで構成されていますが、suffixは識別のためにこちらで勝手に付けたものですので

@honeshabri
Copy link
Member

ANS000516000 については流星戦隊ムスメットの id, title, ruby だけ残せばいいでしょうか?

それでお願いします

@honeshabri
Copy link
Member

honeshabri commented Nov 14, 2016

ルパン三世VS名探偵コナン

「ヴァーサス」でお願いします
調べたところamazonの内容紹介に次のような文があります

“決着(ケリ)つけようぜー。"今世紀最大の宿命の対決(ヴァーサス)!!

Set アヒルノクワック to ruby of ANS000156600
ruby に コンチュウモノガタリ があって id, title に
昆虫物語がないものに追加
それいけ!アンパンマン おともだちシリーズ/ファンタジー
->
それいけ!アンパンマン おともだちシリーズ/ファンタジー「バタコさんと空飛ぶマフラー」
@haruyama
Copy link
Contributor Author

idtitleruby のミスマッチが起きている項目が存在する #20
のコメントをみて commit を足してしまいました. PR を ruby だけに限定したほうがよければ PR を作り直します.

@builtinnya
Copy link
Member

id,title,ruby のミスマッチが起きている項目が存在する #20
のコメントをみて commit を足してしまいました. PR を ruby だけに限定したほうがよければ PR を作り直します.

構いません.ありがとうございます.
こちら WIP でしょうか?そうでなければマージしようと思います.

@haruyama
Copy link
Contributor Author

ルパン三世 VS 名探偵コナン THE MOVIE も ヴァーサス が適当と思うので変更します.
ルパン三世VS名探偵コナン THE MOVIE - Wikipedia でも ヴァーサスですね

これを push したら WIP をはずします

ルパンサンセイ バーサス メイタンテイコナン ザムービー
->
ルパンサンセイ ヴァーサス メイタンテイコナン ザムービー
@haruyama haruyama changed the title [WIP] ルビの修正 ルビの修正 / id, title の整理(#20対応) Nov 14, 2016
@builtinnya
Copy link
Member

@haruyama
長丁場お疲れ様です.
大変助かりました.ありがとうございます 🙇
では,マージいたします

@builtinnya builtinnya merged commit af9eede into anilogia:master Nov 14, 2016
@builtinnya builtinnya added the bug label Nov 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants