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

Poker NPC #112

Closed
merung opened this issue Nov 30, 2018 · 3 comments
Closed

Poker NPC #112

merung opened this issue Nov 30, 2018 · 3 comments
Assignees
Labels
feature This issue adds a new feature to the game.

Comments

@merung
Copy link
Contributor

merung commented Nov 30, 2018

Re-implement vilarion's poker NPC, adjusted to the new module system, and changed lua functions

@merung merung added this to the Update December 2018 milestone Nov 30, 2018
@estralis estralis added the feature This issue adds a new feature to the game. label Dec 1, 2018
@estralis estralis removed this from the Update December 2018 milestone May 19, 2019
@EviePryler
Copy link
Contributor

Tested with Slightly the poker NPC in development server. First thing I noticed was lack of response to triggers like hello, name, job. That might be in the works also think npc needs something telling you how much big and small blinds are, that things are counted in silver (I think this was it), and what the basic rules of the game are (or somewhere in game to find this). I tried using emotes to answer the prompts as a normal rp'r would do. I had issues getting the npc to acknowledge any raise put in an emote. Emotes for call and check worked. I need to test more on those factors that require a number also. Is there anyway to make it freer on word choice?

@Quirkily
Copy link
Contributor

Tested the poker NPC in the development server with EviePryler... First thought... I could grow addicted to this!

Concerns:
1- The first cards shown... These are to determine who will deal, high card deals. Nothing says this. I thought the card was the first card in my hand and then I was asked to place a blind. It was confusing, especially since it seemed to disappear from my hand. Along with this, non poker players won't know what the red chip moving around the table is for either. That though, can be learned in game from other players.
2 - Emotes for raise... "/me raises to 1000000" does not work. It did not like the 's' at the end of raise.
3 - I never knew the total of the pot until the end, then I don't even know what it is unless I win. Ideally, a player should know every round exactly how much is in the pot. After Melissa tells us what cards are available, something should also be posted to each player about the total amount in the pot at that time.
4 - Normally, when playing poker, we can see roughly how much each player has in front of them. I didn't even keep track of how much I had. I'm not sure this can or needs to be changed, but thought I should mention it.

It's actually a lot of fun, but there is so much text scrolling that it becomes confusing rather fast... and this is with just two players. I have to wonder if it will be worse with more playing. Add to that a need to actually know your seat number and pay attention to those numbers being called. Then, add players talking and emoting and it could be difficult. Especially for those not speaking in their native language. Because of that, I think we need a large group test.

@brightrim
Copy link
Member

https://discord.com/channels/401855954272124940/693091825405263893/998912699343843419

  • Lacking immersion
  • Too much text spam
  • Would likely require a custom GUI to actually work properly
  • Too many unresolved issues (as seen in comments above) with no easy solution
  • Delays other features of the Hemp Necktie Underground from being made available

Hence resulting in #388

Potential to be reopened later if we 1. have a dev willing to put in the work and 2. the tools to feasibly even make a poker system work without all the flaws this one sadly has

@brightrim brightrim closed this as not planned Won't fix, can't repro, duplicate, stale Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This issue adds a new feature to the game.
Projects
None yet
Development

No branches or pull requests

6 participants