Takafumi Arakaki
Takafumi Arakaki
> Generally I think we should be preferring log record structure which is logger backend-independent as much as possible. From this point of view putting the "message parts" into what...
> Though thinking about composition, "message and summary" is a distinct concept from Progress, so I'd be somewhat inclined to throw a "thing which is the message and summary" in...
There's already [`Transducers.withprogress`](https://juliafolds.github.io/Transducers.jl/dev/reference/manual/#Transducers.withprogress) which uses ProgressLogging. So, something like this already "works": ```julia using ThreadsX using Transducers: withprogress ThreadsX.sum(x -> gcd(x, 42), withprogress(1:2^30); basesize = 2^20) ``` It's not very...
It sounds like a good addition. I think a straightforward implementation would be to add a new type like `ProgressLogging.Progress`. Something like `Convergence`? `Descent`/`Ascent`? I think the surface syntax requires...
How about `Descent`/`Ascent`? Do we want to support only the minimization case? Optim has [`maximize`](https://github.com/JuliaNLSolvers/Optim.jl/blob/6b01fcd94365721ab451ee0a816af258d6000dbb/src/maximize.jl#L11) function so I thought it might make sense to have different directions. Another possibility is...
We can have multiple `Descent` instances with different IDs. So the progress monitor UI should be able to show different convergence criteria if we only define one or two convergence...
If they share the same parent ID, I suppose it's not too hard for the UI to figure out the relationships? Though it's appealing that we can reduce `shouldlog` overhead...
Hmm... I wonder if it is better to somehow combine this to table log monitor https://github.com/c42f/TerminalLoggers.jl/pull/8  The idea is that each "column" can have independent threshold value,...
@PhilipVinc You need to pass `log_step_increment=0` to log multiple objects in one iteration, right? It may be what @c42f meant by "accumulate." "Vectorized" message type is indeed more TensorBoardLogger.jl friendly....
It's probably better to add `step` field, as it may be costly to log it for each iteration.