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

Amulet of the Acrobat, without the Wu Jian Council nerf #18

Closed
wants to merge 5 commits into from

Conversation

Patashu
Copy link

@Patashu Patashu commented Mar 2, 2018

Letting Amulet of the Acrobat interact with WJC martial arts attacks seems like a pretty Gooncrawl thing. And if it turns out to be broken we can try partial nerfs, for example to 5-8 EV instead of the full 15.

cbuchananhowland and others added 4 commits March 2, 2018 19:38
This amulet boosts the EV of the wearer by 15 only during move actions.
The boost only applies during the execution of the move action, though the
enhanced EV is visible until the player takes another action.

The amulet must be worn at full hp before it begins to operate.

The thought process here is that there's interesting design space around
adding effects to basic actions. The boost size is intentionally quite large
to attempt to balance out the amulet doing nothing during turns spent
fighting.
Since otherwise people will be incentivized to move and forth while waiting.

Also, some light clean-up on the initial Acrobat commit and actually
enable the amulet to spawn.
Wu Jian isn't exactly OP ATM. If this turns out to be too much it can be reduced to +7, +5 or even removed.
@Patashu
Copy link
Author

Patashu commented Mar 2, 2018

Wow, this one is baffling. I guess I better reproduce it...

EDIT: It happens because waiting takes no time. ... Huh.

EDIT 2: It was fixed in another commit, I just didn't notice because it wasn't labelled 'acrobat'.

Broken in bc573f3, which made waiting take neither time nor a turn.
@Floodkiller
Copy link
Owner

I'm not sure if I cherry-picked the commits in the right order as I did not need to apply your specific commit, so I guess we'll just see and revert/reapply if it ends up broken. Closing this for now anyway, as the actual relevant change compared to DCSS's version is pretty easy to remember.

@Floodkiller Floodkiller closed this Jul 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants