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

ゆきぽのCVに落合祐里香を追加 #315

Closed
wants to merge 2 commits into from
Closed

ゆきぽのCVに落合祐里香を追加 #315

wants to merge 2 commits into from

Conversation

oden
Copy link

@oden oden commented Oct 28, 2019

CVが変更されたアイドルの扱いと、時期によって芸名の異なる声優の扱いに対する問題提起的なPRです。

ゆきぽの声優に落合祐里香を追加
@foooomio foooomio mentioned this pull request Oct 28, 2019
@crssnky
Copy link
Member

crssnky commented Oct 29, 2019

Require branches to be up to date before mergingをONにしたので、良きタイミングで最新ブランチに追従よろしくおねがいします!(おそらくUpdate Branchボタンで良いかと)

@foooomio
Copy link
Member

foooomio commented Oct 29, 2019

雪歩の CV に落合祐里香さんを追加すること自体は良いのではないかと思います。

ただ、プルリクの方法で追加すると、例えばこのようなクエリを書いたときに 14レコード取れてしまう問題があります。データとしては正しいんですが、使いやすいデータかというと微妙なところだと思います。

PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>
PREFIX imas: <https://sparql.crssnky.xyz/imasrdf/URIs/imas-schema.ttl#>
SELECT ?idol ?cv
WHERE {
  ?s rdfs:label ?idol ;
    imas:cv ?cv ;
    imas:Title "765AS"@en .
  FILTER(lang(?cv) = 'ja')
}

-------------------------
| idol    | cv          |
=========================
| "水瀬伊織"  | "釘宮理恵"@ja   |
| "双海亜美"  | "下田麻美"@ja   |
| "如月千早"  | "今井麻美"@ja   |
| "菊地真"   | "平田宏美"@ja   |
| "四条貴音"  | "原由実"@ja    |
| "萩原雪歩"  | "落合祐里香"@ja  |
| "萩原雪歩"  | "浅倉杏美"@ja   |
| "双海真美"  | "下田麻美"@ja   |
| "三浦あずさ" | "たかはし智秋"@ja |
| "高槻やよい" | "仁後真耶子"@ja  |
| "秋月律子"  | "若林直美"@ja   |
| "我那覇響"  | "沼倉愛美"@ja   |
| "星井美希"  | "長谷川明子"@ja  |
| "天海春香"  | "中村繪里子"@ja  |
-------------------------

(萩原雪歩の行が2つあることに注目してください)

これを解決する方法として、以下の2つを提案します。

  • imas:formerCv のような述語を新しく定義する。(目的語は xsd:langString や下の schema:Role のどちらでも構わないと思います)
(省略)
    <imas:formerCv xml:lang="ja">落合祐里香</imas:cv>
    <imas:formerCv rdf:resource="http://ja.dbpedia.org/resource/落合祐里香"/>
    <imas:formerCv rdf:resource="http://www.wikidata.org/entity/Q541307"/>
    <imas:cv xml:lang="ja">浅倉杏美</imas:cv>
    <imas:cv rdf:resource="http://ja.dbpedia.org/resource/浅倉杏美"/>
    <imas:cv rdf:resource="http://www.wikidata.org/entity/Q8191313"/>
(省略)

(こちらも試しに書いてみようとしたんですが、声優さんをインスタンス化しないと難しそうです。現在の声優さんを imas:VoiceActor のようなクラスに、過去の声優さんを schema:Role を挟んで振り分ける)

@mpg-yuho-oda
Copy link

master追従しまんた!

@crssnky
Copy link
Member

crssnky commented Nov 2, 2019

主語を分けるか、統一して中で分岐するか、僕の中で天秤が釣り合って決められないんですけど、みなさんはどうでしょうか。
分けるとアイドルの主語を涼ちんみたいに、接尾語を付けます。
統一すると身長体重とかも中で分岐させる予定です。

意見募集です

@foooomio
Copy link
Member

foooomio commented Nov 2, 2019

アーケード版では年齢や身長体重も異なることを忘れていました。アーケード版やゼノグラシアで主語を分けたほうがいいと思います。

Slack であげていた例では imas:cv はそのままで、schema:Role を目的語とするトリプルを追加することを想定していました。しかし、身長体重まで違ってくるとなると影響範囲が広く、現状動いているクエリが動かなくなることが想定されます。

(Slack であげていた例)

imasrdf:Hagiwara_Yukiho a imas:Idol ;
  schema:performIn [
    a schema:Role ;
    imas:title imasrdf:765AS_1st ;
    imas:cv imasrdf:Ochiai_Yurika
  ] ;
  schema:performIn [
    a schema:Role ;
    imas:title imasrdf:765As_2nd ;
    imas:cv imasrdf:Asakura_Azumi
  ] ;
  schema:performIn [
    a schema:Role ;
    imas:title imasrdf:ZENOGLOSSIA ;
    imas:cv imasrdf:Horie_Yui
  ] ;

同じアイドルなのに別のインスタンスになってしまいますが、これは owl:sameAsschema:sameAs で同一人物であることを表現できればと思いますがいかがでしょうか?

@crssnky
Copy link
Member

crssnky commented Nov 2, 2019

1st Visionと2nd Vision、そしてゼノ、果たしてsameAsなんでしょうか(アイマス哲学)

@foooomio
Copy link
Member

foooomio commented Nov 2, 2019

rdfs:seeAlso でも何もつけなくてもいいと思います(笑) sameAs でないという解釈があるのなら、なおさら別インスタンスにするのがいいと考えます。

@crssnky
Copy link
Member

crssnky commented Nov 2, 2019

"インスタンス"という意味を考えると、やはり別が正しいかもしれませんね。
@oden さん問題提起ありがとうございました。おかげさまで、長年放置されてた #61 #229 の解決に目処が立ちました。

@crssnky crssnky closed this Nov 2, 2019
@oden oden deleted the patch-1 branch November 2, 2019 17:48
@oden
Copy link
Author

oden commented Nov 2, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants