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.

Sign up for GitHub

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

Jump to bottom

GetCellValue () got a lost the value of the precision #1373

Closed
zjfGH opened this issue Oct 19, 2022 · 1 comment
Closed

GetCellValue () got a lost the value of the precision #1373

zjfGH opened this issue Oct 19, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@zjfGH
Copy link

zjfGH commented Oct 19, 2022

t.xlsx

value, _ := file.GetCellValue("Sheet1", "A1")
fmt.Println(value)

expect console

88888666665555500000

actual console

-9223372036854775808
@zjfGH zjfGH changed the title GetCellValue () to get the value loss of accuracy GetCellValue () got a lost the value of the precision Oct 19, 2022
@xuri xuri added bug Something isn't working in progress Working in progress labels Oct 19, 2022
@xuri xuri closed this as completed in 2df615f Oct 19, 2022
@xuri xuri removed the in progress Working in progress label Oct 19, 2022
@xuri
Copy link
Member

xuri commented Oct 20, 2022

Thanks for your issue, I have fixed it. Please upgrade to the master branch code, and this patch will be released in the next version.

xuri added a commit to JDavidVR/excelize that referenced this issue Jul 11, 2023
@xuri
This close qax-os#1373, fixes the incorrect build-in number format ap…
4bed2e9 
…ply the result

- An error will be returned when setting the stream row without ascending row numbers, to avoid potential mistakes as mentioned in qax-os#1139
- Updated unit tests
jenbonzhang pushed a commit to jenbonzhang/excelize that referenced this issue Oct 22, 2023
@xuri
This close qax-os#1373, fixes the incorrect build-in number format ap…
920ec7b 
…ply the result

- An error will be returned when setting the stream row without ascending row numbers, to avoid potential mistakes as mentioned in qax-os#1139
- Updated unit tests
@wenchy wenchy mentioned this issue Oct 12, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xuri @zjfGH