machinalis / iepy

Information Extraction in Python
BSD 3-Clause "New" or "Revised" License
906 stars 186 forks source link

Consider removing title and url from document model #58

Closed rafacarrascosa closed 9 years ago

rafacarrascosa commented 9 years ago

That, consider removing title and url fields from iepy.data.models.IEDocument

jmansilla commented 9 years ago

I like it. And even more, I'm thinking that entire metadata field should be removed too. It's adding extra weight to the database for no gain inside iepy.

francolq commented 9 years ago

I agree about removing title and url. I would keep the metadata. In our application, we currently use it to store information that is used in our own document navigation application.

rafacarrascosa commented 9 years ago

EDIT: Already proposed a few minutes ago in #64

Mmm... would it be possible to replace metadata and such by adding a 1-to-1 model wrt IEDocument?

That way we could remove it and encourages our users to do this "trick" the same way Django does with the User model.

j0hn commented 9 years ago

closing this ticket so we follow the discussion on #64