feat: upgrade to [email protected] and implement support for `log(x…

…: Fraction, base: Fraction)`

package.json

@@ -29,7 +29,7 @@
     "complex.js": "^2.2.5",
     "decimal.js": "^10.4.3",
     "escape-latex": "^1.2.0",
-    "fraction.js": "^5.0.4",
+    "fraction.js": "^5.2.1",
     "javascript-natural-sort": "^0.7.1",
     "seedrandom": "^3.0.5",
     "tiny-emitter": "^2.1.0",

src/function/arithmetic/log.js

@@ -2,9 +2,9 @@ import { factory } from '../../utils/factory.js'
 import { logNumber } from '../../plain/number/index.js'
 
 const name = 'log'
-const dependencies = ['config', 'typed', 'divideScalar', 'Complex']
+const dependencies = ['config', 'typed', 'typeOf', 'divideScalar', 'Complex']
 
-export const createLog = /* #__PURE__ */ factory(name, dependencies, ({ typed, config, divideScalar, Complex }) => {
+export const createLog = /* #__PURE__ */ factory(name, dependencies, ({ typed, typeOf, config, divideScalar, Complex }) => {
   /**
    * Calculate the logarithm of a value.
    *
@@ -32,12 +32,12 @@ export const createLog = /* #__PURE__ */ factory(name, dependencies, ({ typed, c
    *
    *    exp, log2, log10, log1p
    *
-   * @param {number | BigNumber | Complex} x
+   * @param {number | BigNumber | Fraction | Complex} x
    *            Value for which to calculate the logarithm.
-   * @param {number | BigNumber | Complex} [base=e]
+   * @param {number | BigNumber | Fraction | Complex} [base=e]
    *            Optional base for the logarithm. If not provided, the natural
    *            logarithm of `x` is calculated.
-   * @return {number | BigNumber | Complex}
+   * @return {number | BigNumber | Fraction | Complex}
    *            Returns the logarithm of `x`
    */
   return typed(name, {
@@ -65,6 +65,15 @@ export const createLog = /* #__PURE__ */ factory(name, dependencies, ({ typed, c
 
     'any, any': typed.referToSelf(self => (x, base) => {
       // calculate logarithm for a specified base, log(x, base)
+
+      if (typeOf(x) === 'Fraction' && typeOf(base) === 'Fraction') {
+        const result = x.log(base)
+
+        if (result !== null) {
+          return result
+        }
+      }
+
       return divideScalar(self(x), self(base))
     })
   })

test/unit-tests/function/arithmetic/log.test.js

@@ -7,6 +7,7 @@ const mathPredictable = math.create({ predictable: true })
 const complex = math.complex
 const matrix = math.matrix
 const unit = math.unit
+const fraction = math.fraction
 const log = math.log
 
 describe('log', function () {
@@ -93,6 +94,13 @@ describe('log', function () {
     approxDeepEqual(log(complex(1, 0)), complex(0, 0))
   })
 
+  it('should return the log of a Fraction', function () {
+    approxDeepEqual(log(fraction(27, 8), fraction(9, 4)), fraction(3, 2))
+    assert.throws(() => log(fraction(27, 8), fraction(-2, 5)),
+      /Cannot implicitly convert a Fraction to BigNumber or vice versa/
+    )
+  })
+
   it('should handle complex number with large imaginary part', function () {
     const tau4 = math.tau / 4
     const real = [0, -1, 1]