search

jl94x4 / ColleXions

ColleXions automates the process of pinning collections to your Plex home screen, making it easier to showcase your favorite content. With customizable features, it enhances your Plex experience by dynamically adjusting what is displayed, randomly.
MIT License
31 stars 1 forks source link

number_of_collections_to_pin isn't used correctly #5

Closed d3mystified closed 1 month ago

d3mystified commented 1 month ago 
Traceback (most recent call last):
  File "/app/ColleXions.py", line 183, in <module>
    main()
  File "/app/ColleXions.py", line 173, in main
    collections_to_pin = filter_collections(config, all_collections, special_collections)
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/ColleXions.py", line 149, in filter_collections
    if len(collections_to_pin) < config['number_of_collections_to_pin']:
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: '<' not supported between instances of 'int' and 'dict'

You need to do config['number_of_collections_to_pin'][library_name].

Is the author even using this script at all?

defluophoenix commented 1 month ago

can you show your config.json? I just ran the script and had no issues

d3mystified commented 1 month ago

I'm using the latest docker image. Seems like it's 10 days old.

jl94x4 commented 1 month ago

I'm using the latest docker image. Seems like it's 10 days old.

Sorry about that, I'll get round to updating it this evening!

jl94x4 commented 1 month ago

@d3mystified the docker is now up to date