-
Notifications
You must be signed in to change notification settings - Fork 4
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
[Bug]: Deck suggestion is not working. #50
Comments
The suggest deck works best for two-color decks. I've found that if I'm playing three or more, it won't give a suggestion because I don't have enough cards for a pure two-color deck. |
That is interesting, and it is a reasonable deck. I would like to know if the data for the color pairs isn't the best after the first day. I've been drafting using the tool for the new set and haven't seen anything strange. I did sealed the first day before data was available, so I don't see your other issue that I just pushed. |
@unrealities, I added my sealed log entry to test_log_scanner so you can check if the format matches the entry from your event. The entry may change depending on the number of active events (for example, 'Course' may change to 'Courses'). It’s probably a good idea to open your sealed log in version 3.27 and consider adding your entry to test_log_scanner to confirm that my change didn’t break anything. |
It looks like the deck suggestion is ignoring Room cards, likely related to this Nevermind, looks like they are included in suggestions. Maybe since the 3.27 update? |
@brobson, version 3.27 didn't change anything regarding the suggestions; it only fixed an issue with the sealed event. I haven't had any issues receiving suggestions that included room cards, but I've also been using the Premier Draft, All Groups dataset for all events. Is this what you're using? |
@FiYir I have been using Premier Draft, All Groups as well. I was thinking that the rooms might be determined by their total mana value and so they are lower weighted due to curve calculations. Meat Locker // Drowned Diner, Underwater Tunnel // Slimy Aquarium, and Grand Entryway // Elegant Rotunda were not being recommended for my White/Blue decks. I edited my post to add the second line about 3.27 because right after I commented here I had Glassworks // Shattered Yard and Ticket Booth // Tunnel of Hate suggested for my Green/Red deck. It seems to be that the rooms are just lower winrate than I though for White/Blue, kicking them out of the suggestions. |
Improvements are needed for high CMC cards that can be played at lower costs, including overlords.
Here's a snippet of Scryfall JSON for Unholy Annex: {
"name": "Unholy Annex // Ritual Chamber",
"mana_cost": "{2}{B} // {3}{B}{B}",
"cmc": 8.0
} Also, recently, many non-creature spells are producing bodies. So not only is this card excluded because of its high CMC, but it is also not considered a creature. This card is a CMC 5 creature and a CMC 3 enchantment. We can modify the code to parse |
Describe the bug in detail
Everything works well , i also dont click before the draft pick end but still deck suggestion is not working , how i can fix it?
Steps to reproduce this bug
Start draft , end pick time and then click the button "Suggest Deck"
Operating System
Windows
Operating System Version
Windows 11
Run Type
Windows Executable
Python Version
No response
Debug Log Details
No response
Possible Solution (optional)
No response
The text was updated successfully, but these errors were encountered: