aim icon indicating copy to clipboard operation
aim copied to clipboard

Add epoch to pytorch_lightning adaptor

Open we-taper opened this issue 2 years ago • 1 comments

🚀 Feature

Add epoch to aim's pytorch_lightning adaptor, whenever it's available.

Pitch

Often the current epoch number is available as a metric from logs sent by pytorch_lightning. If we can extract this epoch to pass it to Aim's logger, then people can set x axis as epoch in Aim's metric explorer. This helps comparing logged values across different experiments with possibly different batch sizes.

Additional context

I am interested to implement this (should be as simple as adding epoch=metric.get('epoch', None) in the adaptor), but I am not sure if this is interesting for you guys? Also, I'd think it's better to keep 'epoch' inside metric still (not del metric['epoch'] after getting it).

we-taper avatar Jul 09 '22 18:07 we-taper

Hey @we-taper thanks for reporting, sorry for the late reply. 🙏 Would you be interested to contribute this one?

SGevorg avatar Jul 11 '22 08:07 SGevorg