rework `check_training_data()` to only check

[EmilHvitfeldt]

To be done after #1330

it is too messy that check_training_set() modifies the data.

in my world check_ functions should only check.

it further complicates things because check_training_set() is where we turn things into tibbles if they aren’t already, and that is confusing