Merge pull request #71 from baumgold/70-InexactError

#70 fix bug preventing large floats from being displayed

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,18 @@ function _showtable_async!(w, schema, names, types, rows, coldefs, tablelength,
  onimport(w, handler)
 end
 
+function _is_javascript_safe(x::Integer)
+ min_safe_int = -(Int64(2)^53-1)
+ max_safe_int = Int64(2)^53-1
+ min_safe_int < x < max_safe_int
+end
+
+function _is_javascript_safe(x::AbstractFloat)
+ min_safe_float = -(Float64(2)^53-1)
+ max_safe_float = Float64(2)^53-1
+ min_safe_float < x < max_safe_float
+end
+
 # directly write JSON instead of allocating temporary dicts etc
 function table2json(schema, rows, types; requested = nothing)
  io = IOBuffer()
@@ -273,7 +284,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