search

Jertzukka / dlensExporter

Python GUI program to convert .dlens into .csv format.
MIT License
12 stars 6 forks source link

TypeError: 'NoneType' #7

Open TatakaRuhito opened 9 months ago

TatakaRuhito commented 9 months ago

When I start the exporter it stops after my first 11 cards (out of 1711). In the command line it says the following:

Preparing files, this might take a bit... Traceback (most recent call last): File "main.py", line 241, in startexport File "main.py", line 196, in getcarddatabyid TypeError: 'NoneType' object is not subscriptable

How can I fix this?

It stops scanning after it has scanned 2 of my Jumstart Terrarion cards. After that it should find 7 copies of Silent dart (I checked which order it scanned in by checking the order in the DelverLens app) Nothing special about the card, just 7 English Silent Dart's from Guilds of Ravnica. I am on Windows 11, installed the latest apk and scryfall json.

TatakaRuhito commented 9 months ago

I deleted silent dart from my list and exported it into a new Dlens file. It has now been abled to scan 371 cards. I think it has trouble scanning cards that Delver Lens has identified as being from the Ravnica Remstared Set. While the cards are actualle from the GUilds of Ravnica set, the same art is used, so it just picks the latest set. I will now sort thorugh my Delver Lens list for al Ravnica Remstered Cards, and change them to Guilds of Ravnica.

Jertzukka commented 9 months ago

@TatakaRuhito Are you using version 1.2, as that had fixes on the Windows unicode formatting? If so, can you provide the .dlens file in question so I can look at it and try to reproduce?

TatakaRuhito commented 9 months ago

Hi,

I found out it's just Cards labbeled to be from Ravnica Remastered that produce the error. Most likely they are not included in the json, of maybe they cause an error in the code. I updated the Dlens file so it's now the correct set Ravnica Allegiance is mentioned, I deleted the old one though.

Will probably be fixed when the new set is officially released and the json is updated.

On Fri, Jan 5, 2024, 22:58 Jertzukka @.***> wrote:

@TatakaRuhito https://github.com/TatakaRuhito Are you using version 1.2, as that had fixes on the Windows unicode formatting? If so, can you provide the .dlens file in question so I can look at it and try to reproduce?

— Reply to this email directly, view it on GitHub https://github.com/Jertzukka/dlensExporter/issues/7#issuecomment-1879303357, or unsubscribe https://github.com/notifications/unsubscribe-auth/APYOJEFQFGA7RVLS2RJGTHDYNBZP7AVCNFSM6AAAAABBOU3VYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZZGMYDGMZVG4 . You are receiving this because you were mentioned.Message ID: @.***>