Update TensorboardLogger

Use hasproperty instead of isdefined on prototypes

bump version to 0.1.13

bump ProtoBuf required version to 0.10
fix project

Author: PhilipVinc

.gitignore

@@ -5,4 +5,4 @@ docs/build
 test/test_logs
 
 gen/proto
-gen/protojl
+gen/protojl

Project.toml

@@ -1,7 +1,7 @@
 name = "TensorBoardLogger"
 uuid = "899adc3e-224a-11e9-021f-63837185c80f"
 authors = ["Filippo Vicentini <filippovicentini@gmail.com>"]
-version = "0.1.12"
+version = "0.1.13"
 
 [deps]
 CRC32c = "8bf52ea8-c179-5cab-976a-9e18b702a9bc"
@@ -18,7 +18,7 @@ ColorTypes = "0.7, 0.8, 0.9, 0.10"
 FileIO = "1"
 FixedPointNumbers = "0.6, 0.7, 0.8"
 ImageCore = "0.7, 0.8"
-ProtoBuf = "0.9.1"
+ProtoBuf = "0.10"
 Requires = "0.5, 1"
 StatsBase = "0.27, 0.28, 0.29, 0.30, 0.31, 0.32, 0.33"
 julia = "0.7, 1"

compat/Project.toml

@@ -1,7 +1,7 @@
 name = "TensorBoardLogger"
 uuid = "899adc3e-224a-11e9-021f-63837185c80f"
 authors = ["Filippo Vicentini <filippovicentini@gmail.com>"]
-version = "0.1.12"
+version = "0.1.13"
 
 [deps]
 CRC32c = "8bf52ea8-c179-5cab-976a-9e18b702a9bc"
@@ -18,7 +18,7 @@ ColorTypes = "0.7, 0.8, 0.9, 0.10"
 FileIO = "1"
 FixedPointNumbers = "0.6, 0.7, 0.8"
 ImageCore = "0.7, 0.8"
-ProtoBuf = "0.7, 0.8"
+ProtoBuf = "0.10"
 Requires = "0.5, 1"
 StatsBase = "0.27, 0.28, 0.29, 0.30, 0.31, 0.32, 0.33"
 julia = "0.7, 1"

src/Deserialization/deserialization.jl

@@ -94,15 +94,15 @@ end
 Returns the type of a summary
 """
 function summary_type(summary)
-    if isdefined(summary, :histo)
+    if hasproperty(summary, :histo)
         return :histo
-    elseif isdefined(summary, :image)
+    elseif hasproperty(summary, :image)
         return :image
-    elseif isdefined(summary, :audio)
+    elseif hasproperty(summary, :audio)
         return :audio
-    elseif isdefined(summary, :tensor)
+    elseif hasproperty(summary, :tensor)
         return :tensor
-    #elseif isdefined(summary, :simple_value)
+    #elseif hasproperty(summary, :simple_value)
     end
     # always defined
     return :simple_value
@@ -212,7 +212,7 @@ function map_summaries(fun::Function, logdir; purge=true, tags=nothing, steps=no
         for event in event_file
             # if event.summary is not defined, don't bother processing this event,
             # as it's probably a "start file" event or a graph event.
-            !isdefined(event, :summary) && continue
+            !hasproperty(event, :summary) && continue
 
             step = event.step
             steps !== nothing && step ∉ steps && continue
@@ -239,7 +239,7 @@ When the keyword argument `purge==true`, if the i+1-th file begins with a purge
 at step `s`, the i-th file is read only up to step `s`.
 
 Also metadata events, without any real data attached are mapped.
-You can detect those by `isdefined(event, :summary) == false`
+You can detect those by `hasproperty(event, :summary) == false`
 
 Optional kwargs `steps` takes as input a collection of integers, and will
 only iterate across events with step within that collection.

src/Deserialization/histograms.jl

@@ -1,6 +1,6 @@
 function deserialize_histogram_summary(summary)
     # custom deserialization
-    if isdefined(summary, :metadata)
+    if hasproperty(summary, :metadata)
         if summary.metadata.plugin_data.plugin_name == TB_PLUGIN_JLARRAY_NAME
             val = reshape(summary.histo.bucket,
                           reinterpret(Int,

src/Deserialization/tensor.jl

@@ -1,3 +1,5 @@
+using .tensorboard: TextPluginData
+
 function deserialize_tensor_summary(summary)
     metadata    = summary.metadata
     plugin_data = metadata.plugin_data

src/Loggers/LogAudio.jl

@@ -1,3 +1,5 @@
+using .tensorboard: Summary_Audio
+
 """
     log_audios(logger::TBLogger, name::AbstractString, samples::AbstractArray, samplerate::Real; step=step(logger))
 

src/Loggers/LogHistograms.jl

@@ -1,3 +1,5 @@
+using .tensorboard: HistogramProto
+
 """
     log_histogram(logger, name, (bins,weights); step=step(logger))
 

src/Loggers/LogImage.jl

@@ -1,3 +1,5 @@
+using .tensorboard: Summary_Image
+
 function image_summary(name::AbstractString, img::PngImage)
 
     attr    = img.attr

src/Loggers/LogText.jl

@@ -1,3 +1,5 @@
+using .tensorboard: TextPluginData
+
 """
     log_text(logger::TBLogger, name::String, text::Any; step=step(logger))