-
Notifications
You must be signed in to change notification settings - Fork 13
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
large number of "Unknown card in collection: Pool does not contain (Will fetch it from Scryfall)" warnings. #29
Comments
The cards are fetched directly from Arena by python-mtga module. Seems like some Arena update changed something that breaks this. I will try to check this more deeply. |
Has been fixed in the lastes Windows executable release. For Linux you have to install python-mtga from pak21/python-mtga repo until his PR is merged into the original repo (mtgatracker/python-mtga#29). |
@robertbarrett Does it work for you now? |
I still face the same issue when using mtga-export.py, but not with the executable. As mtga, I am using your fork:
|
The mtga module from PyPi still does not have the fix for some arena
changes. You need to manually install my fork of python-mtga module from my
repo. Please check the README.
Dňa so 22. 8. 2020, 14:39 ElLorans <[email protected]> napísal(a):
… I still face the same issue:
>python mtga-export.py --goldfish -f mtga_collection_goldfish.csv Found
data @ C:\Program Files\Wizards of the Coast\MTGA\MTGA_Data\Downloads\Data
C:\Program Files\Wizards of the Coast\MTGA\MTGA_Data\Downloads\Data
WARNING! Could not dynamically generate card sets. Do you have Arena
installed? Info: Unknown card in collection: Pool does not contain 70141
(Will fetch it from Scryfall) Info: Unknown card in collection: Pool does
not contain 70267 (Will fetch it from Scryfall) Info: Unknown card in
collection: Pool does not contain 70202 (Will fetch it from Scryfall) Info:
Unknown card in collection: Pool does not contain 70380 (Will fetch it from
Scryfall) Info: Unknown card in collection: Pool does not contain 70163
(Will fetch it from Scryfall) Info: Unknown card in collection: Pool does
not contain 70386 (Will fetch it from Scryfall) Info: Unknown card in
collection: Pool does not contain 70334 (Will fetch it from Scryfall) Info:
Unknown card in collection: Pool does not contain 70275 (Will fetch it from
Scryfall) Info: Unknown card in collection: Pool does not contain 70379
(Will fetch it from Scryfall)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#29 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADI77WFRZ5H4TGVDQK4GGX3SB64BNANCNFSM4PYM55KQ>
.
|
I was using your fork. I solved by using the fork from mtgatracker/python-mtga@4ed584b |
@ElLorans Sorry, I misread your comment. Yes, you are right. I forgot to mention in the README that you need to install the logging branch from my fork. Master in my fork is identical to master in original repo. |
When I run the executable, I get a large output with info messages saying "Unknown card in collection: Pool does not contain
The program eventually does complete succesfully. Is there supposed to be a local db of id's? If I rerun the program against the same player.log file, i get all the same warnings again.
The text was updated successfully, but these errors were encountered: