These are chat archives for PHPOffice/PhpSpreadsheet

Nov 2017
Mike Dodge
Nov 16 2017 04:50
I was getting this warning using PHPExcel, so decided to switch to PHPSpreadsheet to see if I still got the warning, still getting the warning in the same spot
Warning: sqrt() expects parameter 1 to be float, string given in ..\vendor\phpoffice\phpspreadsheet\src\PhpSpreadsheet\Calculation.php
should i open an issue in github?
missed the last part, Calculation.php on line 3761
i'm just reading a bunch of cells in from a spreadsheet, not sure why it needs the sqrt function?
Adrien Crivelli
Nov 16 2017 08:19
@dmikester1_twitter it's most likely because you have formulas in the cells that you are trying to read. Try to find what cell use SQRT function to better isolate the bug, or try to workaround it with getOldCalculatedValue(). See also a loosely related answer:
Adrien Cohen
Nov 16 2017 13:34
@PowerKiKi I started taking a look at the issue, from what I understood we can approach the issue from 2 side, one more for developers which would be 0 based because execpt .xlsx all the other have 0 based column indexes or more for excel users because function like COLUMN() are 1 based
@PowerKiKi currently all the functions of Worksheet class are 0 based. So I would be glad on to help you on that issue but I'm kinda uncertain about the choice 0 based/1 based