TensorBoardLogger.jl icon indicating copy to clipboard operation
TensorBoardLogger.jl copied to clipboard

`log_custom_scalar(lg, layout)` not working

Open rakeshksr opened this issue 2 years ago • 1 comments

Custom Scalars plugin produces an error.

Minimal working example:

julia> using TensorBoardLogger

julia> lg = TBLogger("logs", tb_overwrite)
TBLogger:
    - Log level     : Info
    - Current step  : 0
    - Output        : D:\Downloads\Video\logs
    - open files    : 1


julia> layout = Dict("Cat" => Dict("Curve" => (tb_multiline, ["Curve/1", "Curve/2"])))
Dict{String, Dict{String, Tuple{TensorBoardLogger.tb_chart_type, Vector{String}}}} with 1 entry:
  "Cat" => Dict("Curve"=>(tb_multiline, ["Curve/1", "Curve/2"]))

julia> log_custom_scalar(lg, layout)
ERROR: MethodError: no method matching TensorBoardLogger.tensorboard_plugin_custom_scalar.MultilineChartContent(; tag::Vector{String})

Closest candidates are:
  TensorBoardLogger.tensorboard_plugin_custom_scalar.MultilineChartContent(::Vector{String}) got unsupported keyword argument "tag"
   @ TensorBoardLogger C:\users\user\.julia\packages\TensorBoardLogger\pDb23\src\protojl\tensorboard\plugins\custom_scalar\tensorboard\layout_pb.jl:12
  TensorBoardLogger.tensorboard_plugin_custom_scalar.MultilineChartContent(::Any) got unsupported keyword argument "tag"
   @ TensorBoardLogger C:\users\user\.julia\packages\TensorBoardLogger\pDb23\src\protojl\tensorboard\plugins\custom_scalar\tensorboard\layout_pb.jl:12

Stacktrace:
  [1] chart(name::String, metadata::Tuple{TensorBoardLogger.tb_chart_type, Vector{String}})
    @ TensorBoardLogger C:\users\user\.julia\packages\TensorBoardLogger\pDb23\src\Loggers\LogCustomScalar.jl:28
  [2] #72
    @ .\none:0 [inlined]
  [3] iterate
    @ .\generator.jl:47 [inlined]
  [4] collect(itr::Base.Generator{Base.Iterators.Zip{Tuple{Base.KeySet{String, Dict{String, Tuple{TensorBoardLogger.tb_chart_type, Vector{String}}}}, Base.ValueIterator{Dict{String, Tuple{TensorBoardLogger.tb_chart_type, Vector{String}}}}}}, TensorBoardLogger.var"#72#73"})
    @ Base .\array.jl:782
  [5] charts
    @ C:\users\user\.julia\packages\TensorBoardLogger\pDb23\src\Loggers\LogCustomScalar.jl:42 [inlined]
  [6] (::TensorBoardLogger.var"#74#75")(::Tuple{String, Dict{String, Tuple{TensorBoardLogger.tb_chart_type, Vector{String}}}})
    @ TensorBoardLogger .\none:0
  [7] iterate
    @ .\generator.jl:47 [inlined]
  [8] collect(itr::Base.Generator{Base.Iterators.Zip{Tuple{Base.KeySet{String, Dict{String, Dict{String, Tuple{TensorBoardLogger.tb_chart_type, Vector{String}}}}}, Base.ValueIterator{Dict{String, Dict{String, Tuple{TensorBoardLogger.tb_chart_type, Vector{String}}}}}}}, TensorBoardLogger.var"#74#75"})
    @ Base .\array.jl:782
  [9] custom_scalar_summary(layout::Dict{String, Dict{String, Tuple{TensorBoardLogger.tb_chart_type, Vector{String}}}})
    @ TensorBoardLogger C:\users\user\.julia\packages\TensorBoardLogger\pDb23\src\Loggers\LogCustomScalar.jl:47
 [10] log_custom_scalar(logger::TBLogger{String, IOStream}, layout::Dict{String, Dict{String, Tuple{TensorBoardLogger.tb_chart_type, Vector{String}}}}; step::Nothing)
    @ TensorBoardLogger C:\users\user\.julia\packages\TensorBoardLogger\pDb23\src\Loggers\LogCustomScalar.jl:20
 [11] log_custom_scalar(logger::TBLogger{String, IOStream}, layout::Dict{String, Dict{String, Tuple{TensorBoardLogger.tb_chart_type, Vector{String}}}})
    @ TensorBoardLogger C:\users\user\.julia\packages\TensorBoardLogger\pDb23\src\Loggers\LogCustomScalar.jl:19
 [12] top-level scope
    @ REPL[8]:1

julia>

Version Information:

julia> versioninfo()
Julia Version 1.9.3
Commit bed2cd540a (2023-08-24 14:43 UTC)
Build Info:
  Official https://julialang.org/ release
Platform Info:
  OS: Windows (x86_64-w64-mingw32)
  CPU: 16 × AMD Ryzen 7 PRO 5850U with Radeon Graphics
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-14.0.6 (ORCJIT, znver3)
  Threads: 1 on 16 virtual cores

Package Status:

(temp) pkg> st
Status `D:\Downloads\Video\temp\Project.toml`
  [899adc3e] TensorBoardLogger v0.1.22

rakeshksr avatar Sep 24 '23 04:09 rakeshksr

Duplicate of #120, can be fixed by merging #119.

ldeso avatar Oct 09 '23 11:10 ldeso