Use Symbolics instead of Unitful.uparse to avoid eval during precom…

…pilation

Project.toml

@@ -27,6 +27,7 @@ Random = "9a3f8284-a2c9-5f02-9a11-845980a1fd5c"
 Setfield = "efcf1570-3423-57d1-acb7-fd33fddbac46"
 StatsBase = "2913bbd2-ae8a-5f71-8c99-4fb6c76f3a91"
 SymPy = "24249f21-da20-56a4-8eb1-6a02cf4ae2e6"
+Symbolics = "0c5d862f-8b57-4792-8d23-62f2024744c7"
 TimeZones = "f269a46b-ccf7-5d73-abea-4c690281aa53"
 TypedTables = "9d95f2ec-7b3d-5a63-8d20-e2491e220bb9"
 UnicodePlots = "b8865327-cd53-5732-bb35-84acbb429228"

src/unit.jl

@@ -62,6 +62,7 @@ for f in (:unitfy, :deunitfy)
 end
 
 import Dates
+import Symbolics
 unitfy(df::AbstractDataFrame; kw...) = begin
     #HACK: default constructor for common types to avoid scope binding issue
     D = merge(Dict(
@@ -82,7 +83,10 @@ unitfy(df::AbstractDataFrame; kw...) = begin
             #TODO: remove fallback eval in favor of explict constructor mapping
             haskey(D, e) ? D[e] : Main.eval(e)
         else
-            Unitful.uparse(s)
+            #HACK: use Symbolics to avoid eval during precompilation
+            #Unitful.uparse(s)
+            # https://github.com/PainterQubits/Unitful.jl/issues/649
+            Symbolics.parse_expr_to_symbolic(Meta.parse(s), Unitful)
         end
     end
     u(m) = missing