search

andmarti1424 / sc-im

sc-im - Spreadsheet Calculator Improvised -- An ncurses spreadsheet program for terminal
Other
4.88k stars 208 forks source link

Calculation bug and test7 issue #907

Open DHMike57 opened 1 month ago

DHMike57 commented 1 month ago

Preliminary patch for #906, also addresses #889

I don't fully understand the broken behaviour, so I am not confident about this. It does seem to fix the calculation error (possibly at the expense of too much graph rebuilding and evaluating.

andmarti1424 commented 1 month ago

Thanks for your patch. It might be a little too much to rebuild the entire graph each time a value is entered (let function).

DHMike57 commented 1 month ago

It might be a little too much to rebuild the entire graph each time a value is entered (let function).

I agree, but still better than silently wrong calculations! I was hoping you would know how to come up with something more subtle ;)

andmarti1424 commented 1 month ago

I am afraid no. Not yet at least. I hasnt found where the bug is.