search

nicolaskruchten / pivottable

Open-source Javascript Pivot Table (aka Pivot Grid, Pivot Chart, Cross-Tab) implementation with drag'n'drop.
https://pivottable.js.org/
MIT License
4.35k stars 1.08k forks source link

Sum and Integer Sum Incorrect #1261

Open oguzhangulcan opened 3 years ago

oguzhangulcan commented 3 years ago

Hi Nicolas firstly im realy happy to use this app. thank you. but i have a little problem with sum and integer sum. i'm under limit of 100.000 rows, but sum calculation is less then my total.

can you help me. (i use pivottable from html table)

Thank you..

nicolaskruchten commented 3 years ago

You'll have to provide me with a runnable example so I can reproduce this behaviour.

oguzhangulcan commented 3 years ago

i have long data how can i sent that in excel file to you?

nicolaskruchten commented 3 years ago

Generally folks will use JSFiddle or CodePen or something.

oguzhangulcan commented 3 years ago

probably i found the problem. koli column has got decimals. when pivottable is suming it becoming like column xxx i mean rounddown. :)

bm bsy bayi koli xxx
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 4,000 4,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 10,000 10,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 3,000 3,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 3,000 3,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 29,000 29,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 0,330 0,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 3,000 3,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 3,000 3,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 21,500 21,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 30,000 30,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 468,800 468,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 141,000 141,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 96,000 96,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 271,000 271,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 312,000 312,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 125,600 125,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 22,000 22,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 10,000 10,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 475,550 475,000
DOĞU ANADOLU BÖLGE MÜDÜRLÜĞÜ 16424 ERZURUM * BSY AKGÜN GIDA.PAZ.İNŞ.TAAH.NAK.SAN.VE.TİC.LTD.ŞTİ 148,000 148,000
oguzhangulcan commented 3 years ago

How can i fix this ?

oguzhangulcan commented 3 years ago

hi again :) i have fixed my problem like; i use my data decimal seperator comma but your wonderful script gets with dot seperator firstly i convert my sql query result with using replace comma to dot then pass it to pivot table and in your script i did decimal sep comma again so finally i show on data what i want :)

But if you can do short way to do that, will be better than my way :D 👍

markdeca commented 3 years ago

hi again :) i have fixed my problem like; i use my data decimal seperator comma but your wonderful script gets with dot seperator firstly i convert my sql query result with using replace comma to dot then pass it to pivot table and in your script i did decimal sep comma again so finally i show on data what i want :)

But if you can do short way to do that, will be better than my way :D +1

Hi, I got your same problem so I managed to find a solution without modifying my data. I changed this string in the pivot.js file: return this.sum += parseFloat(record[attr]); to: return this.sum += parseFloat(record[attr].replace(",", ".")); and now seems to work smoothly.