thepaperpilot/Profectus
1
0
Fork 0
forked from profectus/Profectus
Code Pull requests Activity Actions

Fixed isInvertible and isIntegrable not working nested correctly

Browse source
This commit is contained in:
thepaperpilot 2023-04-30 11:08:43 -05:00
parent ff16397cc7
commit bffc27344a
2 changed files with 29 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/game/formulas/formulas.ts
View file

@ -124,11 +124,14 @@ export abstract class InternalFormula<T extends [FormulaSource] | FormulaSource[
const innermostVariable = numVariables === 1 ? variable?.innermostVariable : undefined; const innermostVariable = numVariables === 1 ? variable?.innermostVariable : undefined;
const invertible = variable?.isInvertible() ?? false;
const integrable = variable?.isIntegrable() ?? false;
return { return {
inputs, inputs,
internalEvaluate: evaluate, internalEvaluate: evaluate,
internalInvert: invert, internalInvert: invertible ? invert : undefined,
internalIntegrate: integrate, internalIntegrate: integrable ? integrate : undefined,
internalIntegrateInner: integrateInner, internalIntegrateInner: integrateInner,
applySubstitution, applySubstitution,
innermostVariable, innermostVariable,

28
tests/game/formulas.test.ts
View file

@ -491,10 +491,17 @@ describe("Inverting", () => {
expect(formula.invert(100)).compare_tolerance(0); expect(formula.invert(100)).compare_tolerance(0);
}); });
test("Inverting with non-invertible sections", () => { describe("Inverting with non-invertible sections", () => {
const formula = Formula.add(variable, constant.ceil()); test("Non-invertible constant", () => {
expect(formula.isInvertible()).toBe(true); const formula = Formula.add(variable, constant.ceil());
expect(formula.invert(10)).compare_tolerance(0); expect(formula.isInvertible()).toBe(true);
expect(() => formula.invert(10)).not.toThrow();
});
test("Non-invertible variable", () => {
const formula = Formula.add(variable.ceil(), constant);
expect(formula.isInvertible()).toBe(false);
expect(() => formula.invert(10)).toThrow();
});
}); });
}); });
@ -619,6 +626,19 @@ describe("Integrating", () => {
const formula = Formula.pow(1.05, variable).times(100).pow(0.5); const formula = Formula.pow(1.05, variable).times(100).pow(0.5);
expect(() => formula.evaluateIntegral()).toThrow(); expect(() => formula.evaluateIntegral()).toThrow();
}); });
describe("Integrating with non-integrable sections", () => {
test("Non-integrable constant", () => {
const formula = Formula.add(variable, constant.ceil());
expect(formula.isIntegrable()).toBe(true);
expect(() => formula.evaluateIntegral()).not.toThrow();
});
test("Non-integrable variable", () => {
const formula = Formula.add(variable.ceil(), constant);
expect(formula.isIntegrable()).toBe(false);
expect(() => formula.evaluateIntegral()).toThrow();
});
});
}); });
describe("Inverting integrals", () => { describe("Inverting integrals", () => {