Wrong loader length if overfit=True

[Nimrais]

🐛 Bug Report

runner.train( ... overfit=True )

The runner.loader_sample_len is wrong currently we will have loader length from the original loader. But we must have the first batch size * a number of batches.

How To Reproduce

Take any loader and after runner.train check the train loader length with overfit=True

Expected behavior

we must have the first batch size * a number of batches.

Environment

Not needed

Checklist

• [x] bug description
• [ ] steps to reproduce
• [x] expected behavior
• [x] environment
• [ ] code sample / screenshots

FAQ

Please review the FAQ before submitting an issue:

• [x] I have read the documentation and FAQ
• [x] I have reviewed the minimal examples section
• [x] I have checked the changelog for main framework updates
• [x] I have read the contribution guide
• [x] I have joined Catalyst slack (#__questions channel) for issue discussion

[ditwoo]

I think in the current implementation it's not a bug - it's a feature) Batch overfit is a functionality that allows you to check fast that the model can learn on the same data or not.

[github-actions[bot]]

Hi! Thank you for your contribution! Please re-check all issue template checklists - unfilled issues would be closed automatically. And do not forget to join our slack for collaboration.

[Scitator]

We still should count the sample size in a right way ;)

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[Scitator]

should be fixed ;)