search

cwoac / thingy_grabber

Script for archiving thingiverse things
MIT License
39 stars 10 forks source link

IndexError: list index out of range #5

Closed joebywan closed 4 years ago

joebywan commented 4 years ago

Looking for old dir at ./stls/agentsmith99\GrotMeks-Trakta-Gun-Warehammer-40K Copying 0 unchanged files. Downloading 11 new files of 11 Downloading 13 images. Downloading license Downloading thing 9 - 3170298 [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] [0, 22, 2018] Looking for old dir at ./stls/agentsmith99\Grot-Hanomag-Warhammer-40K Traceback (most recent call last): File "D:\waste\thingygrabber\thingy_grabber.py", line 660, in <module> main() File "D:\waste\thingygrabber\thingy_grabber.py", line 648, in main Designs(user, args.directory, args.quick).download() File "D:\waste\thingygrabber\thingy_grabber.py", line 241, in download RC = Thing(thing).download(self.download_dir) File "D:\waste\thingygrabber\thingy_grabber.py", line 433, in download new_last_time = new_file_links[0].last_update IndexError: list index out of range

This was done using thingy_grabber.py -d .\stls\ user agentsmith99

That's not the only user I'm getting it on. Just incase you think it's an isolated case. AntomanElven\Lucius-Pattern-head-for-Mars-Warlord-Titanicus-scale is another. So far I've gotten it about 4 times today on different ones.

cwoac commented 4 years ago

interesting, so it thinks the model has been updated, but none of the files have newer timestamps. Possibly a readme change or something?

joebywan commented 4 years ago

In my list of 150 odd users to grab it's happening quite frequently.

On Mon, Apr 6, 2020 at 10:39 PM Oliver Matthews notifications@github.com wrote:

interesting, so it thinks the model has been updated, but none of the files have newer timestamps. Possibly a readme change or something?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/cwoac/thingy_grabber/issues/5#issuecomment-609768932, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABE665ZWTOD2FRKS5DHC6ETRLHEQLANCNFSM4L7T2AVA .

cwoac commented 4 years ago

Turned out to be an actual bug rather than just another thingiverse edge case - parsing dates was broken for things where all the files were in October, after the 9th (would also have been wrong for some others, but wouldn't have failed). Fixed by 1267e583c51d66001bf9aeac090ddd37132ee027