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

add speedydex mod #30941

Merged
merged 1 commit into from May 29, 2019

Conversation

Projects
None yet
5 participants
@KorGgenT
Copy link
Contributor

commented May 29, 2019

Summary

SUMMARY: Mods "Add speedydex mod"

Purpose of change

The speedydex mod was one of the mods that was lua only. A couple of people were asking about it, so I took a look at it. turns out it was pretty simple!

Describe the solution

Adds two external options. One controls the minimum dex required to increase your moves, the second one is how many moves increased per point of dexterity. As such, it increases linearly. Then, i added an override to get_speed() in Character that adds that bonus to moves (always 0 if vanilla).
A side affect of this approach is that NPCs will have a higher speed if their dexterity is higher.

@Lorith

This comment has been minimized.

Copy link
Contributor

commented May 29, 2019

And the side effect is actually a bonus that I had wanted with the LUA version but couldn't figure out, win/win with that. Thanks a lot for looking into this.

@dissociativity

This comment has been minimized.

Copy link
Contributor

commented May 29, 2019

if inversely, dex below 8 reduces move speed this might actually balance strength minmaxed characters a bit and a slightly nerfed version might fit into mainline?

@Lorith

This comment has been minimized.

Copy link
Contributor

commented May 29, 2019

That is actually an interesting idea, I think it might fit even better like that. Configure a 'neutral' dex than both above and below that value influence speed instead of just above.

@Mecares

This comment has been minimized.

Copy link
Contributor

commented May 29, 2019

I really like the idea it would at least make str vs dex a more interesting choice. Still would leave per and int on the weak side of stats but an improvement of the current situation nonetheless.

@kevingranade kevingranade merged commit cdf0456 into CleverRaven:master May 29, 2019

2 of 3 checks passed

gorgon-ghprb Build triggered for merge commit.
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@kevingranade

This comment has been minimized.

Copy link
Member

commented May 29, 2019

This pull request has been mentioned on Cataclysm: Dark Days Ahead. There might be relevant details there:

https://discourse.cataclysmdda.org/t/speedy-dex-lua-mod/15344/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.