ProjetPP / Documentation

Documentation and protocol specification of the Projet Pensées Profondes
Creative Commons Zero v1.0 Universal
7 stars 1 forks source link

Adds sort, first and last #34

Closed Tpt closed 9 years ago

Tpt commented 9 years ago

@yhamoudi @Ezibenroc : Please review

progval commented 9 years ago

You only added sort in the description of the serialization

Tpt commented 9 years ago

@ProgVal You should read more carefully the diffs ;-)

progval commented 9 years ago

I did

progval commented 9 years ago

oh… oops…

yhamoudi commented 9 years ago

it would be nice to have more generic connectors than first/last... For example, how can i get the 3rd president of France?

Tpt commented 9 years ago

So, something like position(l, i) that would returns the ith element of l?

I think we should have first and last and maybe implement position later and make first(l) an alias for position(l, 1) et last(l) an alias of position(l, -1). But if you (ping @s-i-newton ) think that it would make the data model to heavy I'm ok to remove first and last and to implement position.

yhamoudi commented 9 years ago

yes, position(l,i) would be good, but it can be implemented later.

(i agree with the datamodel, you can merge)

Ezibenroc commented 9 years ago

Do you expect the Question Parsing modules to produce the sort constructor with a sentence such as Who is the first president of the United States??

yhamoudi commented 9 years ago

i don't if it's a question for me, but yes

Ezibenroc commented 9 years ago

No a question for Thomas. I know that we will never produce it...

yhamoudi commented 9 years ago

? it's easy (no joke)

yhamoudi commented 9 years ago

image

we get all US president with the subtree of root president. Then the node FIRST builds the sort node

Tpt commented 9 years ago

@yhamoudi @Ezibenroc Feel free to merge the pull request if you are ok with it

Ezibenroc commented 9 years ago

Yes but you don't know what we sort. Birth date? Death date? First names?

Ezibenroc commented 9 years ago

Ok for me.

yhamoudi commented 9 years ago

um, yes it's true

progval commented 9 years ago

Default to no predicate, and let Wikidata handle it?

yhamoudi commented 9 years ago

yes, i think wikidata could find a default predicate (the first date it finds for example)

Ezibenroc commented 9 years ago

:+1:

yhamoudi commented 9 years ago

so how to allow connector with no predicate in the datamodel (default predicate)?

yhamoudi commented 9 years ago

In the example:

+{

The predicate is missing?

Tpt commented 9 years ago

"i think wikidata could find a default predicate": possible but far from easy. And I don't think to have time to support such thing before the presentation.