xsanisty/calx.js

Firefox: "too much recursion" while checking for circular references

ncote opened this issue · 2 comments

ncote commented

The call stack is full of cell.fx.checkCircularReference. Rerunning with checkCircularReference: false, fails similarly on cell.fx.processDependency.

There are about 4500 cells, defined in the data: property, not on DOM elements. No circular dependencies, I think (the original Excel didn't have any). Perhaps I've just reached a natural limit? Could dependency handling be flattened, i.e. turned into an iterative rather than recursive algorithm?

Here: http://spreadsheet.aviscuriosa.com/browserscript-calx2.0.html

ikhsan017 commented

It seems a great idea and maybe calculation will be faster, but, I think the process to make dependency tree flattened will need more effort to implement.

ashish8488 commented

Hello @ikhsan017 ,
I am also facing the same error message "too much recursion". But in my case, it does not matter If I make circular references as true or false.

One additional thing:
When I make circular reference to true, then the value in the cell where I have written formula gets blank. I get the calculated values when I make circular reference to false.

Please look into it asap.

Thank you so much for this library. It is really helping me.

Regards,
Ashish Shah