Skip to content
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

Open
cundas opened this issue Sep 28, 2024 · 8 comments
Open

[Bug]: Deck suggestion is not working. #50

cundas opened this issue Sep 28, 2024 · 8 comments
Labels
bug Something isn't working

Comments

@cundas
Copy link

cundas commented Sep 28, 2024

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

@cundas cundas added the bug Something isn't working label Sep 28, 2024
@unrealities
Copy link
Owner

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.

@FiYir
Copy link

FiYir commented Oct 3, 2024

I drafted a bunch of mediocre cards early in DSK, and I also wasn't getting any results from the suggester. I was able to get suggestions once I lowered the ratings_threshold in config.json from 500 to 100, restarted the app, and opened the draft log.

image
image

@unrealities
Copy link
Owner

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.

@FiYir
Copy link

FiYir commented Oct 4, 2024

@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.

@brobson
Copy link

brobson commented Oct 5, 2024

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?

@FiYir
Copy link

FiYir commented Oct 6, 2024

@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?

@brobson
Copy link

brobson commented Oct 6, 2024

@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.

@unrealities
Copy link
Owner

Improvements are needed for high CMC cards that can be played at lower costs, including overlords.

  • Overlord of the Mistmoors (68.9% GIHWR) has a CMC of 7 but can be played for 4.
  • Unholy Annex // Ritual Chamber (65.6% GIHWR) has a CMC of 8 but can be played for 3.

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 mana_cost instead of cmc, but accounting for it being a creature seems much more difficult.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants