Data4Democracy / discursive

Twitter topic search and indexing with Elasticsearch
21 stars 11 forks source link

ISSUE #2: Adds TOPICS configuration to textfile #5

Closed jbrambleDC closed 7 years ago

jbrambleDC commented 7 years ago

@hadoopjax This addresses Issue #2 . I used sys.argv for simplicity, but if you would like me to import argparse and use ArgParsing on the command line we can do that instead. Another separate option would be to use config package and define the filepath in the config.py. Which do you think works best? I think the deciding factor is whether we prefer the flexibility of command line parsing. or the definitiveness of specifying a path in the config.py

Thoughts?

hadoopjax commented 7 years ago

Personally, I think this method works great for folks that aren't going to be comfortable putting terms into CLI. Basically, with this, you could have a list of terms you wanted to use, clone/download the repo, drop your topics.txt into the cloned repo folder and be more-or-less off and running (assuming you've setup all your supporting credentials, infra).