catalyst: WandB batch metrics logging error

🐛 Bug Report

In wandb all batch metrics are logged as single value per epoch.

Expected behavior

Batch metrics must be logged once per step.

Catalyst version: 21.7

Additional context

The problem is here:

https://github.com/catalyst-team/catalyst/blob/master/catalyst/loggers/wandb.py#L115

Step must be equal to global_sample_step, not global_epoch_step.

About this issue

  • Original URL
  • State: closed
  • Created 3 years ago
  • Comments: 20 (18 by maintainers)

Most upvoted comments

Just running tests. PR coming soon

Okay, thanks for the clarification. I’ll update the logger with the intended artifacts use case.

@Scitator sounds good. I’ll do it within 2 days

Thank you for your response! May be it is better to always log sample_step?

  • @Scitator I’m out on a vacation. I’ll have a look at this on monday. But it sounds like we might not have a good solution for this as W&B only has one global step.