Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v0.23.0 worksheet_formula not reading majority of formulas #384

Closed
luisdlt9 opened this issue Dec 14, 2023 · 2 comments · Fixed by #388
Closed

v0.23.0 worksheet_formula not reading majority of formulas #384

luisdlt9 opened this issue Dec 14, 2023 · 2 comments · Fixed by #388

Comments

@luisdlt9
Copy link

calamine = "0.23.0" is not reading the majority of formulas on workbooks. On some workbooks it is not reading any formulas at all. As a simple example in the attached workbook I have 3 simple formulas scattered around, v0.23.0 only picks up a single formula "C1+E5". However, calamine = "0.22.0" picks up the 3 formulas correctly "C1+E5", "SUM(1+1)", "IF(OR(Q22="",Q22="United States"),"US","Foreign")".

The formula issue isn't just with "0.23.0" even "0.22.0", "0.21.0", and "0.20.0" are not picking up a lot of the formulas in my workbooks, however "0.23.0" is exponentially worse at handling formulas, with most formulas not being picked up at all.

formula issue.xlsx

@david-rse
Copy link

Having the same problem I think. When iterating a worksheets rows, given a column with a formula, neither the value nor formula is captured within the &[DataType]. Not experiencing deserialization problems with other non-formula columns. Same issue whether iterating over the rows or iterator derived from RangeDeserializerBuilderas seen in the examples.

@tafia tafia mentioned this issue Dec 19, 2023
@tafia
Copy link
Owner

tafia commented Dec 19, 2023

Should be fixed now.
Sorry for that and thank you for reporting it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants