jamuraa / gatherling

Magic: The Gathering tournament runner and tracker for Pauper Magic
http://pdcmagic.com/gatherling
MIT License
11 stars 11 forks source link

Meddle winning decks bug on player profile #75

Open Dabil opened 12 years ago

Dabil commented 12 years ago

The player profile groups deck by name instead of deck contents. This makes it possible for players to give two completely different decks the same name and have the combined record show incorrectly. Many players have figured this out and use this knowledge to "pump up" the record of their meddle winning decks. And sometimes when they play a deck in the past that has meddled, if it doesn't do well in the current tournament, they give it a different name so it doesn't weaken the record of their meddle winning deck.

Another reason to group by deck contents is so you can use Gatherling to track the success of new cards, as gatherling would then split out decks with different card selection on the player profile page.

jamuraa commented 12 years ago

The original thought behind grouping by name is that I wanted to make it possible to tweak a deck through its lifetime and still track the record, even though it may have been tweaked over time.

I don't know if splitting by unique deck hash will be the desirable result here. We could require that people have a minimum amount of common cards for them to be considered the same. You could find a deck that drifted to be a completely different deck over time, but it seems unlikely. On Sep 12, 2012 9:02 AM, "Dabil" notifications@github.com wrote:

The player profile groups deck by name instead of deck contents. This makes it possible for players to give two completely different decks the same name and have the combined record show incorrectly. Many players have figured this out and use this knowledge to "pump up" the record of their meddle winning decks. And sometimes when they play a deck in the past that has meddled, if it doesn't do well in the current tournament, they give it a different name so it doesn't weaken the record of their meddle winning deck.

Another reason to group by deck contents is so you can use Gatherling to track the success of new cards, as gatherling would then split out decks with different card selection on the player profile page.

— Reply to this email directly or view it on GitHubhttps://github.com/jamuraa/gatherling/issues/75.

Dabil commented 12 years ago

Yeah requiring a certain number if cards in common us a good solution. For deck averaging myself and others use 50 cards in common between the sideboard and main. On Sep 12, 2012 10:26 AM, "Michael Janssen" notifications@github.com wrote:

The original thought behind grouping by name is that I wanted to make it possible to tweak a deck through its lifetime and still track the record, even though it may have been tweaked over time.

I don't know if splitting by unique deck hash will be the desirable result here. We could require that people have a minimum amount of common cards for them to be considered the same. You could find a deck that drifted to be a completely different deck over time, but it seems unlikely. On Sep 12, 2012 9:02 AM, "Dabil" notifications@github.com wrote:

The player profile groups deck by name instead of deck contents. This makes it possible for players to give two completely different decks the same name and have the combined record show incorrectly. Many players have figured this out and use this knowledge to "pump up" the record of their meddle winning decks. And sometimes when they play a deck in the past that has meddled, if it doesn't do well in the current tournament, they give it a different name so it doesn't weaken the record of their meddle winning deck.

Another reason to group by deck contents is so you can use Gatherling to track the success of new cards, as gatherling would then split out decks with different card selection on the player profile page.

— Reply to this email directly or view it on GitHub< https://github.com/jamuraa/gatherling/issues/75>.

— Reply to this email directly or view it on GitHubhttps://github.com/jamuraa/gatherling/issues/75#issuecomment-8495284.