address ability to run without validation

[DrClick]

Before submitting

• Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
• Did you read the contributor guideline?
• Did you make sure to update the docs?
• Did you write any new necessary tests?

What does this PR do?

Fixes #536.

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 🙃

[neggert]

Could we not just check to see if validation_step is defined rather than adding yet another argument to the Trainer constructor?

[DrClick]

The point is, you may have defined it but not want to call it in this particular training run.

[Borda]

I do not see any code change, just formatting whitespaces

[DrClick]

Yes sorry, there are two commits. I did not see the flake8 errors on the first one. Do you want the commits squashed? I appreciate the work on this codebase and want to contribute, I am just getting used to the contribution guidelines.

[williamFalcon]

shouldn’t you just do?

training_step(...):
    return 0

i also don’t think we need another arg

[Borda]

would be easier to set val_check_interval to infinity or -1? instead of adding a new parameter...

[williamFalcon]

To do this you can set

val_percent_check=0.0