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

Fix the Dark Sight spell #35850

Merged
merged 3 commits into from
Dec 5, 2019
Merged

Fix the Dark Sight spell #35850

merged 3 commits into from
Dec 5, 2019

Conversation

CSHague
Copy link
Contributor

@CSHague CSHague commented Dec 4, 2019

Both difficulty and max level of the Dark Sight spell were originally set to zero. This made it impossible to raise the level of Dark Sight. Justifying the value chosen for difficulty by the clairvoyance spell, which gives you instantaneous 1 turn vision on all tiles in the aoe based on level. Clairvoyance has difficulty 4. Whereas Dark Sight gives encumbrance-free light amp vision for the same amount of time as a magical light spell. Compared to the Magical Lamp spell which generates regular light at difficulty 2, for the same amount of time. Comparing the distance you gain in vision from light amp, difficulty 6 seems justified.

Added some mana cost to the dark sight spell to make it equal to magical light.

Summary

SUMMARY: Balance "Fix the Dark Sight spell"

Purpose of change

Noticed while mucking around that Dark Sight had zero difficulty compared to a spell that was more expensive that merely gave you a magical night light for the same period of time. Moving on, it was made clear that the dark sight spell had an unlisted max level value which defaults to zero making it impossible to level .

Describe the solution

Change the difficulty to a value greater than zero. Set max level to 20. Add 100 mana to mana cost.

Describe alternatives you've considered

Difficulty 4 or 5 would be fine too but 6 seemed more appropriate given that the tech solution is cheaper but encumbers you while the magical version has no encumbrance at all.

Testing

Spell trains properly now. Takes longer to learn but not too long.

Additional context

Durations still feel a little short but that's still in-question.

Was originally zero. Justifying the value chosen by the clairvoyance spell, which gives you instantaneous 1 turn vision on all tiles in the aoe based on level. Clairvoyance has difficulty 4. Whereas Dark Sight gives encumbrance-free light amp vision for the same amount of time as a magical light spell. Compared to the Magical Lamp spell which generates regular light at difficulty 2, for the same amount of time. Comparing the distance you gain in vision from light amp, difficulty 6 seems justified.
@ZhilkinSerg ZhilkinSerg added [JSON] Changes (can be) made in JSON Game: Balance Balancing of (existing) in-game features. Mods: Magiclysm Anything to do with the Magiclysm mod labels Dec 4, 2019
@KorGgenT
Copy link
Member

KorGgenT commented Dec 4, 2019

you appear to also be changing the duration increment for dark sight and magical lamp. i also happened to notice that dark sight doesn't appear to have a max level defined for it.

@CSHague
Copy link
Contributor Author

CSHague commented Dec 5, 2019

Did I leave that in there? I thought I reverted that change. That was supposed to be a separate branch. The durations didn't match max level by ratio like the other spells did. It says max duration is ten times the starting duration but at level 20, max, you can only get about half that.

This was supposed to be in another branch
@CSHague
Copy link
Contributor Author

CSHague commented Dec 5, 2019

Okay, I reverted that. This does bring up a question: The min duration and max duration on these spells don't match the rate at which the spell levels. How can I be certain what the original intent was other than basing it on the idea that they wanted the spells to be proportional? Were the durations on the light/sight spells were an oversight? I can't just assume this and change it without any feedback.

You'd need to change the level increment to 45,000 (my original number was off by a factor of 10) to match the max duration (1,000,000) listed in several spells. This doesn't seem right either. Perhaps the maximum level on the lamp and magical light spells should be removed or set higher? That way the player could potentially strive for that 10000 second magical light but it would take a really long time.

@KorGgenT
Copy link
Member

KorGgenT commented Dec 5, 2019

I would say that the scope of this PR is trying to fix obvious problems with Dark Sight. So the other issue i see is that it has a max level of 0, meaning the increments don't do anything at all, regardless of the original intent.

Set it at 20, like every other spell. Made the mana cost equivalent to Magical Light.
@CSHague CSHague changed the title Add Difficulty to Dark Sight Fix the Dark Sight spell Dec 5, 2019
@KorGgenT KorGgenT merged commit d8f47ae into CleverRaven:master Dec 5, 2019
@CSHague CSHague deleted the patch-3 branch December 5, 2019 14:51
I-am-Erk pushed a commit to I-am-Erk/Cataclysm-DDA that referenced this pull request Dec 6, 2019
* Add Difficulty to Dark Sight

Was originally zero. Justifying the value chosen by the clairvoyance spell, which gives you instantaneous 1 turn vision on all tiles in the aoe based on level. Clairvoyance has difficulty 4. Whereas Dark Sight gives encumbrance-free light amp vision for the same amount of time as a magical light spell. Compared to the Magical Lamp spell which generates regular light at difficulty 2, for the same amount of time. Comparing the distance you gain in vision from light amp, difficulty 6 seems justified.

* Add max level and 100 mana cost to Dark Sight

Set it at 20, like every other spell. Made the mana cost equivalent to Magical Light.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Game: Balance Balancing of (existing) in-game features. [JSON] Changes (can be) made in JSON Mods: Magiclysm Anything to do with the Magiclysm mod
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants