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

Slot reward appears that negates a nonexistent risk #657

Open
cannonfodder1 opened this issue Jun 15, 2021 · 6 comments
Open

Slot reward appears that negates a nonexistent risk #657

cannonfodder1 opened this issue Jun 15, 2021 · 6 comments
Assignees
Labels
bug Something isn't working gameplay
Milestone

Comments

@cannonfodder1
Copy link
Contributor

Issue is likely related to the Reapers resistance order that prevents the risk of ambush on all covert actions.
Discord ref: https://discord.com/channels/578606792448409610/639765305345835009/854168036986191884

@cannonfodder1 cannonfodder1 added bug Something isn't working gameplay labels Jun 15, 2021
@cannonfodder1 cannonfodder1 added this to the Pre release milestone Jun 15, 2021
@Xymanek
Copy link
Member

Xymanek commented Jun 17, 2021

In vanilla, optional slots that negate an already negated risk are simply not shown to the player.

Our options:

  • Leave everything as-is. Drawback: player confusion regarding negation of a non-existent risk
  • Hide "negates risk" hint on the slot. Drawback: player confusion regarding the "useless" optional slot and why some CAs have it and others do not
  • Hide slot if risk is already negated (vanilla). Drawback: player doesn't get the opportunity to reduce the other risk(s) as much (ambush rarely (never?) comes alone). This is also the most involved change, and we do not have another extended beta period to verify that it would work properly
  • Remove the Guarding Angels card. Drawback: the situation can resurface due to interactions with other mods

@cannonfodder1 thoughts?

@cannonfodder1
Copy link
Contributor Author

I believe that out of all those options, removing the "Negates Risk: X" banner has the least worst drawback. The optional slot won't be completely useless because additional risk reduction can be gained from it.

@Xymanek
Copy link
Member

Xymanek commented Jun 18, 2021

Looking at the code deeper, turns out that option isn't so simple simple either. The issue is that the reason "why was this negated" isn't saved anywhere, only that it was negated. So while we can easily hide the "negates ambush" on UISS, it would come back on UICAG post-launch and the report screen.

I'm once again considering removing GA, especially considering how powerful it in CI environment, especially for T1 card

@cannonfodder1
Copy link
Contributor Author

GA could possibly be changed to a different effect that doesn't cause the nonexistent counter issue and is easier to implement. One idea might be reducing the risk trigger chance of Ambushes by a certain percentage.

@Xymanek
Copy link
Member

Xymanek commented Jun 18, 2021

Ok, go ahead

@cannonfodder1
Copy link
Contributor Author

Change of plans, Guardian Angels will be removed and this issue will be revisited post-release.

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

No branches or pull requests

2 participants