Fix num_batches with fast_dev_run & multiple test dataloaders

[lebrice]

Fixed a small bug that seems to happen when using fast_dev_run and multiple
test dataloaders.

Signed-off-by: Fabrice Normandin fabrice.normandin@gmail.com

What does this PR do?

Fixes # (issue)

Before submitting

• Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
• Did you read the contributor guideline, Pull Request section?
• Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
• Did you make sure to update the documentation with your changes?
• Did you write any new necessary tests?
• Did you verify new and existing tests pass locally with your changes?
• If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

[lebrice]

Please don't accept this yet, or please convert it to a Draft, I'm still experiencing the issue. Will update this when its fixed
Ok false alarm, this commit does indeed resolve the issue for me after all.

[rohitgr7]

Duplicate #2581?

[codecov]

Codecov Report

Merging #2661 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master   #2661   +/-   ##
======================================
  Coverage      91%     91%           
======================================
  Files          72      72           
  Lines        6129    6129           
======================================
  Hits         5597    5597           
  Misses        532     532           

[lebrice]

Ok false alarm, this commit does indeed resolve the issue for me after all.

@rohitgr7 It does seem like it might be a duplicate of #2581 , with this one being a one-line fix without the proper documentation and tests.
Feel free to close this PR if you want.

[rohitgr7]

Nothing much. Needed to add some test with fast_dev_run, not possible by explicitly changing num_batches with fast_dev_run. Also removed some lines by explicitly changing limit_{mode)_batches in __init__.

[williamFalcon]

duplicate was merged