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

ROUNDDOWN function is not evaluated properly #1189

Closed
4 tasks
iliankostovbulpros opened this issue Sep 19, 2023 · 3 comments · Fixed by #1310
Closed
4 tasks

ROUNDDOWN function is not evaluated properly #1189

iliankostovbulpros opened this issue Sep 19, 2023 · 3 comments · Fixed by #1310

Comments

@iliankostovbulpros
Copy link

NPOI Version

2.5.6

File Type

  • [ x ] XLSX
  • XLS
  • DOCX
  • XLSM
  • OTHER

Upload the Excel File

Rounddown_Issue.xlsx

Reproduce Steps

  1. Load Rounddown_Issue.xlsx - using var fs = new FileStream("Rounddown_Issue.xlsx", FileMode.Open, FileAccess.Read);
  2. Create new instance of workbook - var wb = new XSSFWorkbook(fs);
  3. Get the first sheet - var sheet = wb.GetSheetAt(0);
  4. Create new instance of formula evaluator - var formula = new XSSFFormulaEvaluator(wb);
  5. Evaluate all - formula.EvaluateAll();
  6. Read numeric cell value of G4 - g4.NumericCellValue

Actual output: 17.55
Expected output: 17.56

Issue Description

NPOI formula evaluation does not calculate properly the value of G4 as it is calculated in Excel.
The excel calculation template shows value 17.56 at cell G4, but the NPOI library evaluate it to numeric cell value of 17.55

@Bykiev
Copy link
Collaborator

Bykiev commented Sep 19, 2023

Hi, seems it was already fixed in #1110, check latest version.

@iliankostovbulpros
Copy link
Author

Hi again, we upgraded NPOI to the latest stable 2.6.2 and the issue still exists. Can we ask you to check our repro steps again?

@tonyqus
Copy link
Member

tonyqus commented Sep 20, 2023

@iliankostovbulpros I double checked the code is merged to release2.6.2 branch. Your issue can be a new issue.

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

Successfully merging a pull request may close this issue.

3 participants