#70 fix bug preventing large floats from being displayed

Author: Ben Baumgold

Project.toml

@@ -1,6 +1,6 @@
 name = "TableView"
 uuid = "40c74d1a-b44c-5b06-a7c1-6cbea58ea978"
-version = "0.6.7"
+version = "0.6.8"
 
 [deps]
 Dates = "ade2ca70-3891-5945-98fb-dc099432e06a"

src/TableView.jl

@@ -7,7 +7,6 @@ using Observables: @map
 export showtable
 
 const ag_grid_imports = []
-const js_max_safe_int = Int128(2^53-1)
 
 function __init__()
     version = readchomp(joinpath(@__DIR__, "..", "ag-grid.version"))
@@ -259,6 +258,14 @@ function _showtable_async!(w, schema, names, types, rows, coldefs, tablelength,
     onimport(w, handler)
 end
 
+function _is_javascript_safe(x::Integer)
+    -Int128(2^53)-1 < x < Int128(2^53)-1
+end
+
+function _is_javascript_safe(x::AbstractFloat)
+    -2^53-1 < x < 2^53-1
+end
+
 # directly write JSON instead of allocating temporary dicts etc
 function table2json(schema, rows, types; requested = nothing)
     io = IOBuffer()
@@ -273,7 +280,7 @@ function table2json(schema, rows, types; requested = nothing)
         columnwriter = JSON.Writer.CompactContext(io)
         JSON.begin_object(columnwriter)
         Tables.eachcolumn(schema, row) do val, ind, name
-            if val isa Real && isfinite(val) && -js_max_safe_int < trunc(Int128, val) < js_max_safe_int
+            if val isa Real && isfinite(val) && _is_javascript_safe(val)
                 JSON.show_pair(columnwriter, ser, name, val)
             elseif val === nothing || val === missing
                 JSON.show_pair(columnwriter, ser, name, repr(val))

test/runtests.jl

@@ -42,6 +42,15 @@ end
     @test firstrow["b"] == "9007199254740992"
     @test firstrow["c"] == "18014398509481984"
 end
+@testset "large floats" begin
+    rows = Tables.table([1.0e50 1.0e100])
+    names = [:a, :b]
+    types = [Float64, Float64]
+    json = TableView.table2json(Tables.Schema(names, types), rows, types)
+    firstrow = JSON.parse(json)[1]
+    @test firstrow["a"] == "1.0e50"
+    @test firstrow["b"] == "1.0e100"
+end
 @testset "normal array" begin
     array = rand(10, 10)
     @test showtable(array) isa WebIO.Scope