search

DaylightingSociety / SocMap

Social Mapping Framework for Twitter
https://socmap.daylightingsociety.org/
BSD 3-Clause "New" or "Revised" License
18 stars 4 forks source link

Either implement 'Remove Tweets' or pull the command line option #19

Closed milo-trujillo closed 6 years ago

milo-trujillo commented 6 years ago

Right now the documentation states:

-r, --removetweets    Remove tweets after extracting metadata

That flag does nothing. We should either implement nuking the tweets as we download them (and maybe leave an empty file in place to mark that we've downloaded those tweets?) or remove the command line option if we don't intend to implement the feature.

Does this feature have merit? Are there enough cases where users would want the Tweet maps but not the Tweets, and are so space-constrained that it matters?

I'm leaning towards "No one cares, pull the feature", but it should also be relatively easy to implement, so what does everyone else think?

milo-trujillo commented 6 years ago

Users may frequently want to split the mention and retweet maps, which requires re-processing the downloaded tweets. For now we will pull the option, and let users delete tweets by hand if they want.