Re: [Tracker] TV Series in tracker



Le 2012-04-05 11:59, Ivan Frade a ÃcritÂ:
Hi,

 My comments:

0. It is missing the "season" idea, which is quite important. We could
copy the pattern for the multiple CD albums... although is not the
nicest part of the ontology.
1. TVShow doesn't need to be direct subclass of all those classes.
Inheriting from nfo:Video will pull the rest of the hierarchy
  In other words, the hierarchy now is (starting in the lowest
subclass): nfo:Video, nfo:Visual, nfo:Media, nie:InformationElement

  I wonder if TVShow is a nfo:Video (something "playable") or
nmm:Video (something you want to see in your media player).

I would definitely vote for nmm:Video, a tvshow is not a special file type


2. The classes don't have rdfs:domain (or rdfs:range). That only
applies to properties. (Remove rdfs:domain from nmm:TVShow)
3. nmm:series comment should explain "a TVShow belongs to a serie"
4. Actually I would rename nmm:series to nmm:isEpisode or
nmm:isEpisodeOf, to make it clear it is the reverse property of
nmm:hasEpisode
5. nmm:TVShow comment should explain that it is usually an Episode.
Why don't we call it directly Episode? In a TVShow with independent
stories (E.G. The Twilight Zone), each piece is still call Episode,
right?

 Thanks for starting this work. Hopefully we can give it a couple of
iterations and merge soon.

 Regards,

Ivan



On Wed, Apr 4, 2012 at 11:40 PM, Jens Georg <mail jensge org> wrote:
On Di, 2012-04-03 at 11:24 +0100, Martyn Russell wrote:
On 02/04/12 09:50, Jens Georg wrote:
> On Mo, 2012-04-02 at 09:34 +0100, Martyn Russell wrote:
>
>>> Yes, sure. I'd even be willing to write a small tool to do that manually >>> for now; I just need the ontology to generate the data because I
>>> desperately need this for my private Rygel use-case ;)
>>
>> We'll happily accept patches Jens ;)
>>
> I know that, but I've never touched the ontologies so I'd rather leave
> that to someone who already did this.

I don't much either :)

Anyway, something like the patch attached should work. It passes the
ontology validator. Perhaps Ivan/Philip or JÃrg could comment?

Btw, wouldn't it make sense to set nmm:synopsis to
tracker:fulltextIndexed true ?

_______________________________________________
tracker-list mailing list
tracker-list gnome org
http://mail.gnome.org/mailman/listinfo/tracker-list




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]