engine: do not use keywords as keys

inveniosoftware-contrib / invenio-classifier

Invenio module for record classification.

GNU General Public License v2.0

6 stars 12 forks source link

Closed kaplun closed 7 years ago

kaplun commented 7 years ago

Do not use keywords as dictionary keys, rather as elements in a list. (Closes inspirehep/inspire-next#2456)

Signed-off-by: Samuele Kaplun samuele.kaplun@cern.ch

michamos commented 7 years ago

won't this fail for the same reason it failed with the experiment guesser (inspirehep/inspire-next#2054)?

kaplun commented 7 years ago

Well, I now have to adapt inspire-next to it.

michamos commented 7 years ago

I mean using lists as tuples with different types inside that make ES choke.

kaplun commented 7 years ago

Ha! Now I understand. I am actually further fixing indeed classifier to properly output list objects rather than list of tuples.