Description
Based on the discussion in PR #21, the divisor parameter in the ExponentialCurve class should be removed, and the implementation should revert to dividing by base instead.
Context
- The
divisor parameter was added to allow more control over the curve behavior
- However, this implementation can cause mathematical inconsistencies as pointed out in the discussion
- The current equation is:
y = ((x/abs(x)) * ((1 + [base])^abs(x) -1)) / [divisor]
- The original equation was:
y = ((x/abs(x)) * ((1 + [base])^abs(x) -1)) / [base]
Suggested Changes
- Remove the
divisor parameter from the ExponentialCurve class
- Revert to dividing by
base in the implementation
- Update all usages of
ExponentialCurve in the codebase to remove the second parameter
References
Requested by @Baconing
Description
Based on the discussion in PR #21, the
divisorparameter in theExponentialCurveclass should be removed, and the implementation should revert to dividing bybaseinstead.Context
divisorparameter was added to allow more control over the curve behaviory = ((x/abs(x)) * ((1 + [base])^abs(x) -1)) / [divisor]y = ((x/abs(x)) * ((1 + [base])^abs(x) -1)) / [base]Suggested Changes
divisorparameter from theExponentialCurveclassbasein the implementationExponentialCurvein the codebase to remove the second parameterReferences
Requested by @Baconing