You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
First step: Create a shifting variable that will shift between -1 and 1
Second step: use a wrapper with the target set on the variable and a constant of 1 or -1
Third step: Output the result of the wrapper.
What I expected to happen:
I expected the wrapper to shift between -1 and 1 depending upon the target
What happened instead:
the wrapper only shifted from 0 to 1 (if the constant was 1) or -1 to 0 (if the constant was -1)
there was no way to feed in a range from -1 to 1 and have the wrapper output that range.
The text was updated successfully, but these errors were encountered:
Fixing this would require changing the return type of SpellPiece.evaluate to account for both minimum and maximum values as the metadata calculation of some pieces could be dependent on the sign of a constant parameter.
To properly support this without making assumptions on spell costs that could be violated by an addon introducing tricks with nonstandard inputs that could vary between positive and negative values, it would require an overhaul of the underlying system to allow pieces to output a range at compile time.
Such a change is possible, but I wouldn't do so immediately due to it causing incompatibilities with addons. This change is probably best saved for refactoring while porting to 1.17.
Version Information
Forge version:35.1.37
Psi version:1.16-91
Further Information
Steps to reproduce:
What I expected to happen:
I expected the wrapper to shift between -1 and 1 depending upon the target
What happened instead:
the wrapper only shifted from 0 to 1 (if the constant was 1) or -1 to 0 (if the constant was -1)
there was no way to feed in a range from -1 to 1 and have the wrapper output that range.
The text was updated successfully, but these errors were encountered: