Merge pull request #51 from pie-framework/fix/logarithm-conversion

(fix): logarithm conversion from latex to ast

Author: CarlaCostea

src/conversion/latex-to-ast.ts

@@ -1273,11 +1273,7 @@ export class LatexToAst {
 
         this.advance();
       } else {
-        parameter = this.statement({
-          parse_absolute_value: parse_absolute_value,
-          unknownCommands: unknownCommands,
-        });
-
+        parameter = this.token.token_text;
         this.advance();
       }
 

src/fixtures/latex-equal/symbolic/PD-302.ts

@@ -5,6 +5,7 @@ export default {
       target: "\\log(x*y)",
       eq: [
         "\\log(x)+\\log(y)",
+        "\\log x+\\log y",
         "\\log_{10}(x)+\\log_{10}(y)",
         "\\log_{10}(x*y)",
       ],
@@ -14,6 +15,7 @@ export default {
       target: "\\log(x/y)",
       eq: [
         "\\log(x)-\\log(y)",
+        "\\log x -\\log y",
         "\\log_{10}(x)-\\log_{10}(y)",
         "\\log_{10}(x/y)",
       ],

src/fixtures/latex-equal/symbolic/log-conversion.ts

@@ -6,6 +6,22 @@ export default {
       eq: ["\\log_{10}(x)", "\\log x"],
       ne: ["\\log_{3}(x)"],
     },
+    {
+      target: "\\log (x)",
+      eq: ["\\log_{10}(x)", "\\log x"],
+    },
+    {
+      target: "\\log (y)",
+      eq: ["\\log_{10}(y)", "\\log y"],
+    },
+    {
+      target: "\\log x",
+      eq: ["\\log_{10}(x)", "\\log(x)"],
+    },
+    {
+      target: "\\log x +\\log y ",
+      eq: ["\\log(x) + \\log(y)"],
+    },
     {
       target: "\\ln_{e} (x)",
       eq: ["\\log_{e}(x)", "\\log_{e} x"],

src/latex-equal.ts

@@ -26,9 +26,11 @@ export const latexEqual = (a: Latex, b: Latex, opts: Opts) => {
   }
 
   const al = lta.convert(a);
+  console.log(al, "al")
 
-  const bl = lta.convert(b);
 
+  const bl = lta.convert(b);
+  console.log(bl, "bl")
   if (differenceIsTooGreat(al, bl)) {
     return false;
   }