search

Wordseer / wordseer

The WordSeer text analysis tool, written in Flask.
http://wordseer.berkeley.edu/
42 stars 16 forks source link

Merge pipeline and application models #38

Closed keien closed 10 years ago

abendebury commented 10 years ago

Where should the models live? Now they live inside app, but they're used outside of app as well, so things are getting trickier...

keien commented 10 years ago

We should leave it in app, it doesn't really make sense to have it anywhere else. An option we can consider is moving the pipeline into app/wordseer/pipeline since it's so integrated now that it doesn't really make sense to leave it in lib anymore, plus it'll make imports and tests easier.

abendebury commented 10 years ago

Yes, that's a good idea.

keien commented 10 years ago

Let's finish up all the merges first and then we can do that, unless you'd rather just take care of it now?

abendebury commented 10 years ago

No, let's finish the merging first.