minmay wrote:Serious issues:
So, let's start answering your points! First if I can make a request, when you say something does not work, please give some example of a bug it could trigger, from now on I will try my best to guess what you mean by "is bad", "problem" or "doesn't work", so it is possible I miss the target...
minmay wrote:- Might doesn't really work right, the stat bonuses should be applied in the condition's onRecomputeStats with Champion:addStatModifier(), instead of trying to use Champion:setBaseStat().
Why not, but is it really safe to read the current stat inside the onRecomputeStats function? Or do you have another idea in mind to get the stat value?
minmay wrote:- handling for not meeting energy costs (e.g. reduced cost of disrupt) is really bad. It should act the same as not meeting the energy cost in the builtin spell system. Consider changing it from "energy cost is reduced by X" to "X energy is refunded after casting" which would not suffer the same problem.
That is what I had for those spells in a previous version of the pack. I was not happy with it... So, the only thing I can imagine is different from the builtin spells is about PartyComponent.onCastSpell triggering hooks for non paid spells, is that the problem? (every thing else has builtin similar behavior I think). If so, a solution could also be to report the test to the main onCastSpell hook of spells_functions.lua
minmay wrote:- discharge should be removed since it can break so many puzzles by killing normally-unreachable monsters
Is there really such puzzles in the mod? Is this good puzzle design?
minmay wrote:- air/earth/fire/water enchantment are, frankly, really obtuse and also overpowered. Should be removed.
Okay, would a nerf and some notes in the dungeon about their effects change your mind?
minmay wrote:- All Shall Fall's removal of flying from monsters can also break puzzles, and unless you make a lot of new animations for crowerns etc. it doesn't really make sense in the first place.
I agree it is cheap without new animations, but it has some game play value. This is nearly always prefered to realism in gaming
minmay wrote:- like Discharge, zone effects can be used to affect monsters that are supposed to be unreachable. Not sure how to fix this without doing some fairly complex pathfinding. Unlike discharge I think the ranges are short enough that they don't currently break any rooms, though.
Do you have examples of such rooms that I could test?
minmay wrote:- zone effects also use the wrong geometry to compute whether a tile is within their radius, remember that while Grimrock's space is visually Euclidean, it is functionally Taxicab for the most part (gravity-affected projectiles are an exception, not the norm)
I think you refer to my use of tiles coordinates on non 3 meters module height maps? I should correct that, sure!
minmay wrote:- ice shards should be reverted to the builtin targeting rules like the other burst spells (custom damage rules is fine). You don't actually have to check the ground in front of the party because IceShardsComponent does that automatically; see my recreation of the spell.
Okay, I totally missed this component functionalities when I reproduced the behavior. I can remove it, but from what I tested so far, I managed to get the same behavior as the builtin ice shards. If I remove it, I suppose the ice shards component can be used for other ground attack spells too?
minmay wrote:- your poison cloud implementation doesn't merge CloudSpellComponents like the builtin one, this is important because yours allows stacking a large number of poison clouds. Cloud merging is nontrivial to recreate and you will pretty much have to reimplement CloudSpellComponent with TileDamagerComponent and TimerComponent (to decay the attack power). I think that with rock fall added, earth magic doesn't need poison cloud anyway so it is likely better to remove it.
I never knew this even existed. Is there really a potential FPS problem with stacking clouds? What does the engine do about it, are you talking about merging objects? Because components merging does not make sens to me, unless you talk about components from multiple objects. I did not know the game deleted those objects to protect its framerate in the first place. Is there a plausible scenario in which a player would stack TOO MUCH clouds? (Removing one of the original spells is a bit excessive, isn't it?)
minmay wrote:- I know you had good intentions by removing the firebeam tracer, but you misunderstand its purpose. The tracer is required for the spell to work properly on heightmapped levels; by removing the tracer you have allowed the spell to pass through the ground. (I will fix this next time I get the file).
I left it quoted after testing some spells behaviors with non-integer elevation casting. I stopped working on it when you told me you had a new version of the spell.
minmay wrote:Not bugs, but complaints:
In this part other modders thoughts are very very welcome
minmay wrote:- frostburst spell is too powerful (frostbolt is higher level, is not guaranteed to freeze, and is still an amazingly good spell). A dedicated melee freeze spell could work but it needs to be higher level.
I think we really need a spell with this level of requirement for the water school. Frostburst was evident as it was the only missing burst spell in the game. I thought with a very short freeze effect it would not be so overpowered. Maybe we could reduce its duration? About the necessity of having spells at certains skills level requirements, and especially at this first skill point, see my answer about the goals of the mod below.
minmay wrote:- I still massively disagree with your hailstorm buff against freezing immune targets. If water magic actually gets better against cold immune targets, then what's the point of cold immunity even existing?
Well, first "frozen" immune targets and "cold" damage type immune targets are not the same. Water spells crowd control may be very powerful, but they are always disabled against bosses in all games since the dawn of humanity (don't come with an odd case!). The idea is to give water school some damage when its crowd control is negated. Its cold damage will still be reduced by monsters' resistance. And monsters not immune to its crowd control will still require using the low damage crowd control spells to build a "shatter" effect. May be another spell should do it but not hailstorm, but I think it should be a water spell (or a multi schools spell with water). Also, perhaps this spell should remove the frozen condition on hit, so a storm type spell is not ideal.
minmay wrote:- I do not think spells should give haste, rage, or resurrection - those are supposed to be limited resources and haste is extremely powerful.
I thought rage was the best effect (when stacked)? I don't mind removing haste from wind_rider though, now that it as the air cushion effect. Do you want to remove all buff spells? Also spells are supposed to be powerful, just not too much compared to non-casters champions. Before removing a spell or effect, you can always try to reduce its power.
minmay wrote:- Spells should be learned as soon as the scroll is obtained, instead of making players carry the scroll the first time they cast it. At the very least, use PartyComponent:isCarrying() to determine whether the party has the scroll, instead of making you put it in a specific champion's inventory and remove it from containers. Alternatively, it's easy to change my spellbook to automatically record new spells as the scrolls are picked up.
I totally missed this function! But reading a spell scroll from inside a bag or other champions' pockets is not easy =D
A new way to learn spells is really welcome! Making scroll usable and/or using your spellbook + using PartyComponent:isCarrying() should be nice features. Also, I never had any answer about the spell restriction feature. Perhaps nobody wants it in the first place?
minmay wrote:I still think 111 spells is an insanely high number. That's a ton of cognitive load for players and I really think it should be reduced; here are my ideas:
That is the big point to discuss, and it is totally linked with the kind of gaming experience we all want for this mod. 111 spells is sure way too much for a single player. But how many spells would you like?
To get all the spells you would have to get to at least level 23 with a skilled non farmer human, or use several skill tomes, or multiple casters. I see only two possibilities to get to this point: the player really wants it, and that's cool, or the player has imported a high level party. So this is a very important question to consider before cutting spells; will the mod allow champions import? If the answer is no, then most of the spells are in practice out of reach of the average player, and reducing their number is useless. And still in that scenario, the end game level of a typical caster is also very important.
In the allowed import scenario, another way we could limit the number of spells a player can get would be to limit the number of scrolls the can find/buy, etc... It could be interesting if they could read all the scrolls but only choose something like 20 (for example) during one game.
minmay wrote:1. Get rid of most of the "palette swap" spells - only have a rune trap for one element, only have a meteor storm for one element, probably just get rid of frostburst (ice shards is a much more unique spell, and there's already frostbolt for freezing things), and there's no need for the combined storms or combined shields.
Again depending on allowing import party or not, palette swap is mainly there to neuter the "bad skill choice! no luck!" effect I found awful with Isle of Nex spells that led me to making this pack. Not sure if you will agree, but I see it that way; earth magic was the bad choice trap for uninformed players, as much boring with so few spells as weak compared to real end game magic schools like fire, fire or fire. (Air and Water are debatable, but not for their damage). that must be why so many people here have posted custom spells frequently for the earth magic school. I don't think a game with a leveling system that simple and character customization that limited should have a few good builds and all the rest is garbage. So, the goal was a least one new spell each time the player levels up, and yes similar spells, not very exiting but efficient from a player perspective I think.
minmay wrote:1.5. Alternatively, have just a single "Bolt Storm" spell that includes bolts from the elemental schools you have. So if you only have fire magic 4, it acts like meteor storm, and if you only have air magic 4, it acts like thunder storm, and if you have fire,air,earth,water 4, it acts like elemental storm. Do the same thing for elemental shielding. This reduces 30 overly similar spells to 2 spells.
Why not, but again, learning a new spell when you level up, to me is not the same as your old spell gaining +20% damage. I understand it may be unnatural to you to see it that way, after all learning a new spell from a new scroll that just does 20% more damage than your old spell is in fine the same thing as a passive stat boost, but I believe it is more enjoyable for the average player.
The BIG BIG BIG advantage of elemental storm is not the pretty colors or even damage type considerations, but the fact that it requires 2 skill points in each elemental school, for a total of 8 skill points, while meteor storm requires 5 fire magic + 3 concentraion (was air magic in the builtin version) for the same total. So the player who wanted to play an elementalist and thought it was a good idea will not be punished by the game and be left with only burst and ice shards spells, no projectile spell, at the same level an informed player gets meteor storm.
minmay wrote:2. If the resurrection spells are kept, at least reduce the number of them. I count 7, and the game also has potions of resurrection and crystal shards and big healing crystals. I think breath of life alone is enough, and if not, the 7 resurrection spells could easily be combined as above, but I think just having one of breath of life OR soul bound is cleaner.
Not sure about that. In a way ressurection spells, having high energy cost, are limited by energy potions. But I could also add a longer cooldown to them?. I would say they truly are game breakers in single use crystal iron mode only.
All that being said, I am okay with reducing the spells number, we could even remove all multiple magic schools spells and still respect the rule of at least one new spell for each new skill point. This rule, and having all possible builds nearly similar in power, are the strict minimum I want from the final product
)