feat: cohortdefiners return validatedframes

[sarakolding]

Automatically created PR from invoke

[github-actions]

Looks like your PR modifies shared library files in psycop/common/.

We highly recommend getting your code reviewed by one of the core maintainers to avoid breaking other projects that depend on these files :-)

[sarakolding]

@MartinBernstorff what I think is happening is that the PredictionTimeCollator (in seq models) is no longer runtime_checkable since it contains CohortDefiner, and CohortDefiner now uses the ValidatedFrame which is not runtime_checkable. But I'm pretty lost (and Lasse is too), so maybe we can talk about it tomorrow 🐒

[sarakolding]

(ps it is not done - I can't run the tests because of the error)

[MartinBernstorff]

Sure thing. I'm here today; I'll wait until we can talk 👍

[sarakolding]

thoughts: more cols/validators? e.g., uuid col with a unique rule?
also, maybe we should make a getattribute method or something, so the frame can be called from the instance, instead of the attribute?