Giter Site home page Giter Site logo

Comments (11)

jensb89 avatar jensb89 commented on May 29, 2024

Do you use the latest version? I have fixed a similar bug lately.

If yes, do you know at which series/movie it fails. Then I can try to reproduce and debug.

from netflix-to-trakt-import.

justkorix avatar justkorix commented on May 29, 2024

Yeah it's the latest version, not sure where it's getting knocked out. If you can tell me where I can grab the log I'd be happy to upload it for you to see where the error is coming in

from netflix-to-trakt-import.

jensb89 avatar jensb89 commented on May 29, 2024

There should be a file "Netflix2TraktImportLog.log" in the folder.

Additionally, you could set TMDB-DEBUG to True in the config.py file for more infos in that log file.

Or alternatively you could simply send me your csv file (or part of it) here so I could try to reproduce the error with that. Thanks!

from netflix-to-trakt-import.

Nackophilz avatar Nackophilz commented on May 29, 2024

Same problem here

When this is a big file he cant process trough all (maybe a limitation of api call on TMDB ? )

from netflix-to-trakt-import.

Nackophilz avatar Nackophilz commented on May 29, 2024

This is definitely because of large CSV file.

I add to the python script a function to parse one CSV file into one CSV file of 500 lines max (as many as i need)

then loop on each new csv file to work

And voila ! :)

from netflix-to-trakt-import.

justkorix avatar justkorix commented on May 29, 2024

Any chance I could snag that script? Would love to get the rest added to Trakt!

from netflix-to-trakt-import.

ahknight avatar ahknight commented on May 29, 2024

When I ran into this error it's because it was searching for season "None" and on line 95 it was calling tmdbSeason.details outside of a try block. I added the try block (mirroring the one above it for tmdbSeason.details on line 72) and it skipped over an entry in the CSV that looked like:

ShowName: NameOfFirstHalf / NameOfSecondHalf

It should probably default to searching on season 1 if season is None.

from netflix-to-trakt-import.

jensb89 avatar jensb89 commented on May 29, 2024

Thanks for the updates @ALL.

A few thoughts on this:

  • There should not be a searching for "None" on line 95, because the code there is only executed for season.number != None. So I think that is not a problem. Skipping of some items can happen sometimes though (but rarely due to schema in the csv that is not easy to match)
  • A try block around the TMDB Api call is still a good idea! I will implement that in the coming days
  • The Tmdb Api has no rate limits, but some people in some forums write that their CDNs have some basic rate limits of 50 requests/second (=each 20ms). We have a 100ms sleep time, so should be outside the range. But it seems it might still be a problem
  • The sync to the TraktAPI is already cutting to max 1000 elements for each call, but the TMDB api search is currently done for all items first
  • From what I read, it might be good to break the whole script after around 500 items and sync them to Trakt and then start with the next 500 (maybe after a break of 1-2s)
  • The above idea requires some more restructuring though. When I find some time I will try to change it that way.

For now, it seems the best thing is to split large files into separate smaller files, until the above mentioned fix is ready :)

from netflix-to-trakt-import.

SawkeeReemo avatar SawkeeReemo commented on May 29, 2024

https://hastebin.com/share/ilocahifuc.java

Having this problem as well. It's happening so often, that this is sort of unusable at the moment.
NetflixViewingHistory.csv

from netflix-to-trakt-import.

SawkeeReemo avatar SawkeeReemo commented on May 29, 2024

Quick update: Since I had over 10 years worth of Netflix history to import, I switched to grabbing the more recent stuff first and copying that into a new CSV. I was able to get through most of it, which is good enough for me right now, but if I can help provide more data in any way to help you squash this bug, I'd be happy to. Great script you've written here, must be a pain to parse all this out.

from netflix-to-trakt-import.

jensb89 avatar jensb89 commented on May 29, 2024

Thanks for the updates @SawkeeReemo.
Yes, splitting into smaller CSVs is the solution here currently. It's good to know that it works for you and the code is helpful to many people. In general the code still works fine for files that are not too large. There are still some open points. For example, checking if the same entry was already uploaded would be a nice feature to avoid double entries in trakt. The large file problem is another.
However, some of these require a bigger code change and more testing. I hope to find some time for that soon. Still,
any help here is appreciated :)

from netflix-to-trakt-import.

Related Issues (19)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.