Explicitly declare used functions/types from dependencies (#84)

Author: PhilipVinc

src/Loggers/LogHistograms.jl

@@ -45,7 +45,7 @@ function histogram_summary(name::AbstractString, edges::AbstractVector{<:Number}
     return histogram_summary(name, Histogram(edges, hist_vals))
 end
 
-function histogram_summary(name::AbstractString, hist::StatsBase.Histogram{T,1}) where T
+function histogram_summary(name::AbstractString, hist::Histogram{T,1}) where T
     edges     = first(hist.edges)
     hist_vals = hist.weights
 

src/PNG.jl

@@ -2,7 +2,7 @@
 module PNGImage
 
 using FileIO: @format_str, Stream, save
-using ImageCore: Colorant, Gray, GrayA, RGB, RGBA
+using ColorTypes: Colorant, Gray, GrayA, RGB, RGBA
 using FixedPointNumbers: Normed
 
 export PngImage, png_color_T

src/TensorBoardLogger.jl

@@ -1,16 +1,19 @@
 module TensorBoardLogger
 
-using ProtoBuf, CRC32c
+using ProtoBuf: readproto, writeproto, ProtoType
+using CRC32c
+
+using ImageCore: colorview, channelview
+using ColorTypes: Colorant, Gray, GrayA, RGB, RGBA
+using FileIO: FileIO, @format_str, Stream, save, load
 
 # hasproperty is not defined before 1.2. This is Compat.hasproperty
 if VERSION < v"1.2.0-DEV.272"
     using ProtoBuf: hasproperty
 end
 
-using ImageCore, ColorTypes
-using FileIO: FileIO, @format_str, Stream, save, load
-
-using StatsBase  #TODO: remove this. Only needed to compute histogram bins.
+#TODO: Is there a more lightweight package for compmuting an histogram?
+using StatsBase: Histogram, fit
 
 using  Base.CoreLogging: CoreLogging, AbstractLogger, LogLevel, Info,
     handle_message, shouldlog, min_enabled_level, catch_exceptions

src/logger_dispatch.jl

@@ -50,7 +50,7 @@ end
 ## Default behaviours
 
 ########## For things going to LogImage ##############################
-function preprocess(name,   img::AbstractArray{<:Colorant}, data)
+function preprocess(name, img::AbstractArray{<:Colorant}, data)
     # If it has three dimensions (and we don't have 3D monitors) we log several
     # 2D slices under the same tag so that TB shows a slider along the z direction.
     dimensions = ndims(img)
@@ -76,8 +76,8 @@ summary_impl(name, value::Any) = text_summary(name, value)
 
 ########## For things going to LogHistograms ########################
 # Only consider 1D histograms for histogram plotting
-preprocess(name, hist::StatsBase.Histogram{T,1}, data) where T = push!(data, name=>hist)
-summary_impl(name, hist::StatsBase.Histogram) = histogram_summary(name, hist)
+preprocess(name, hist::Histogram{T,1}, data) where T = push!(data, name=>hist)
+summary_impl(name, hist::Histogram) = histogram_summary(name, hist)
 
 # TODO: maybe deprecate? tuple means histogram (only if bins/weights match)
 function preprocess(name,   (bins,weights)::Tuple{AbstractVector,AbstractVector}, data)