-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cannot filter retweets #13
Comments
I have fixed this on my own branch, but it has slightly deviated from the original codebase, so it cannot be easily merged. Fortunately the changed are simple to replicate, here is the relevant commit: faeinthebay@653cebe I missed some documentation which I fixed in another commit. I apologize for not offering a clean pull request, I'm still learning Git and good practices. Thanks for your time! |
I should also note that I have conducted basic tests on my fork and have found no issues including and excluding retweets. |
i get an error after using the filter retweet option "unrecognized arguments: -w". I did editing the codes correctly from what you provided, so what did it go wrong in my case? |
Thank you so much Andrew. I came across twphotos and was so happy to be able to mirror entire accounts, but upon discovering retweets were being saved as well, it was a huge disappointment, as some twitter accounts contain more retweets than original posts, and thus the original downloads of said account were tainted by random other images from other accounts. I've successfully merged your code and tested it, and it's working great. Thank you so much for the contribution! I've went so far as to make a batch script that reads a txt file that lists a username on each new line, which it parses and runs the command per username. (along with twitter-dl to snatch video posts.) A function like this would be great is to be able to do this without having to close and restart the connection to twitter (thus hitting the twitter api limitation as quickly.) Although that's well out of my coding experience with Python, so I just parse 30-40 accounts to each txt file when updating, as the API resets every 15 minutes. |
I also get the same error when attempting to use the retweet filter option: |
Although the Python twitter API supports filtering out retweets, it is not currently possible to filter them in this utility. I suspect users (including myself) use the user argument to download photos posted by twitter users and might not care about retweeted photos that are not originally theirs.
The text was updated successfully, but these errors were encountered: