btouellette / concarneau

Multiplayer web game using the same rules as Carcassone
MIT License
65 stars 10 forks source link

Precise rules for number of tiles? #9

Closed Ud71p closed 6 years ago

Ud71p commented 7 years ago

Hi, and thanks for this great game.

I think I've noticed some change in rules? I think long time ago there was more tiles than now? I'm not sure.

But I am sure that the number of tiles is being limited, because in some games a rare tile might not come.

In case I'm right that there is some limiting, it would be great to write these rules down, perhaps on a Wiki?

Even better, allow choosing number of tiles when creating a game. This way one could play a quick game, but still with 2 extensions.

btouellette commented 7 years ago

No change to the rules or the tile sets. There's only one situation in which not every tile would be played in a game and it is when a tile is drawn which doesn't have a valid placement on the field. In that case it silently discards the tile and draws another one for the user which could be confusing. I'll see if I can add a message to the game chat when a tile is discarded that way at least it can be verified that every tile was drawn at some point.

Ud71p commented 7 years ago

I see, it makes sense. This explains why I don't always get 2 cathedrals. But what about the starting tile? I have more than 1 in my physical game and I can see on Wikipedia that there is 4 of these. But I have observed that in Concarneau there's only 1: the one the game starts with and then it never comes again? So is the tile distribution different than in Carcassonne?

Ud71p commented 6 years ago

OK, I found the source for tile distribution, it's in tile.js/tileSchema.statics.loadTilesBase. Here in fact RCr.png has count=4, which is correct, but when I play it only appears once. Maybe the bug is that it has startingTile: true, and instead there should be one with startingTile: true and 3 with startingTile: false?

btouellette commented 6 years ago

Tile schema was fine but there was a bug where rather than just loading one less of the startingTile into the set of tiles it was skipped entirely. Fixed in 05e18241f67de373287ccfeba8a6d7a9e35c812a (for any new game created going forward). Thanks for the catch.