Insert logged times into state

🚀 Feature

By default, Engine measures times for single epoch and total run. These times are logged with engine.logger: https://github.com/pytorch/ignite/blob/efc45a79ee0a5bf8105c3caff194758439db2426/ignite/engine/engine.py#L793-L796 and https://github.com/pytorch/ignite/blob/efc45a79ee0a5bf8105c3caff194758439db2426/ignite/engine/engine.py#L497-L500

Idea is to put these times (at least time_taken for epoch and total time) into engine.state under a name (e.g. “times”). Such that we can also use this information without need to use Timer or setup logger.

[vfdev-5]

@erip yes, both can be helpful and I agree with you about

I think it makes most since to just maintain a single value for now and let the user determine if/how they want to maintain that list if necessary.

We can leave to the user the way to store all times, as other values we store in the state, e.g. metrics, batch, output etc.

[vfdev-5]

@erip Actually, we above snippet is wrong. I fixed it.

it would imply that a timer of some sort will need to be instantiated internal to Engine.state, right?

Yes, in run method or even in State by default…

I’m trying to think through how the Δt will be computed in the log_times call above.

Maybe we do not need any deltas to compute as

• time_taken in _run_once_on_dataset will give the time for 1 epoch.
• time_taken in _internal_run will give the time for complete run.

Does it sound good ?