-
Notifications
You must be signed in to change notification settings - Fork 397
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
Recurring Credit Cards are only partially implemented #362
Comments
I can try to take this on as my first task unless there are any objections. I think it would be a good starter task to get me acquainted with the code and how the card definitions work. I'll definitely ask questions as they come up. |
That is indeed the reason! This sounds like a challenging task but definitely doable, and will very much help you learn the code :). |
My thought is to break the recurring credit usage into cases, and then deal with each case separately. From all the recurring credit cards, the following categories surfaced: Runner: Corp: Question: Can anyone think of a situation where a player would NOT want to use their recurring credits first before regular credits? I feel like we could automate this by always using recurring credits for a specific category if they are available, and then deduct from regular pool after that. If we think players might want to micromanage which credits get used, then we'll likely need UI entries for every case. I could see the install case being challenging since it's a click to auto install/deduct, so there'd be new UI flow around that. Thoughts? |
I can think of few times that recurring credits should not be used automatically:
This is sounding more difficult :). An hierarchical categorization might be needed for number 3... spend credits under Keep thinking about a design. If you want to start with something related but a little easier, we (devs) have casually brainstormed about how to implement Bad Publicity's credit in a way that would tie nicely into Stimhack, and could start the foundation for this system. The idea we've tossed around is to set a variable inside |
Thanks for the feedback, Neal. Your points definitely convince me that we shouldn't automate any recurring credits. These use cases definitely point to instances where a player would be frustrated if we automatically spent credits on their behalf. I'll look at Stimhack and Bad Pub and keep thinking about recurring credits, as I think you're right; a lot of these are related and it would be good to think about how they might tie together. |
On a related note, how fully implemented are Stimhack and Bad Pub separately? Stimhack seems to give the 9 creds but I can't see where/if it removes unused credits. Bad Pub seems like a good straight forward starting point for recurring credits. |
You've answered your own question: Stimhack does not remove credits, and bp credit is completely manual. |
I actually quite like the way jinteki.net handles stealth credits. It's much easier to play a stealth deck on jinteki.net than it is on octgn (where recurring credits are used automatically). I think that Bad Pub and stimhack are things that could definitely be improved though. My thoughts about both is that a dummy card could be installed for the duration of a run with either Bad Pub or 9 credits respectively that can be used like recurring credits. As a related point, the feature I would most like to see added is a game state editor, to fix mistakes made by either player. Being able to add/remove tokens from cards would be really handy. Edit: Neal's suggestion about BP is better than mine. Ignore me... |
I think automating recurring stealth credits is gonna be hard. And dont even mention blackat where u have to chose if u use a normal or stealth cred. The current implementation might not be perfect, but works pretty wel imho. On the other hand. Automating scrubber and wizzard doesnt sound to hard. Cant imagine a case where u wouldnt want to use rec. cred first to trash something. |
There are a bunch of edge cases where you would want to use your credit pool even with whizzard/scrubber. I think it's dangerous to automate recurring credits until you can easily put them back so you can choose to not use them retroactively.
I know 95%+ of the time you're going to want to use the recurring credits, but I actually think automating them will be detrimental to the UX of jintiki.net. |
I'm definitely going to make them manual. Same with Bad Pub and Stimhack credits. Ideally I'll make the defaults in the choice prompt max out the values so by default it should be quick to confirm credits, but then allow for manual tinkering if necessary. Sounds good? |
Here's what I'm thinking for recurring credits. Since this is my first big-ish feature I thought I'd put my thoughts in the issue in case people see flaws or better ways to do this. I'm going to give it a run for one of the easier cases where there's existing UI I can co-opt. Currently the recurring credits are just a map of the :recurring keyword to a value. My plan is to change this to point to another map containing the following: Here's what it might look like generally: Then in places we use these recurring credits, we can check the relevant player's hand for the specific type of recurring credit that may exist and augment the UI accordingly to allow for manual spending. Here are some potential examples: "Dedicated Server" "Net Police" "Primary Transmission Dish" "Net Celebrity" "Cyberfeeder" "Cybsoft MacroDrive" "Dyson Fractal Generator" "Cloak" Here's the full list of client changes needed: Credits for installing - will need new UI since this is done automatically. Credits for using traces - can just add to the existing UI. |
Justin, do you want an invite to the developer Slack that was started? On Wed, Aug 5, 2015 at 1:28 PM, Justin Liew [email protected]
|
I would love to, please! |
Let me know at jkoepp -at- gmail.com so I can get your email for sending it! On Thu, Aug 6, 2015 at 10:06 AM, Justin Liew [email protected]
|
Sent. |
Parity with main
Most if not all of the recurring credit cards are only partially implemented. I believe this is because there is no mechanism to filter those recurring credits to specific uses (ie. for using fracters, for rezzing ice).
The text was updated successfully, but these errors were encountered: