Eugen Beck

Sorry, I was not precise enough. What I meant was that in the engine class you know for what the dataset is used and from which name in the config...

OK, let's use `def dataset_post_process(data: TensorDict, *, train: bool = False) -> TensorDict: ...`

3.8 is ok (as it's the default for ubuntu 20.04). Anything higher I would delay for now until we (Apptek) have firmly tested our new ubuntu 22.04 images. Thanks!

Shall we merge @michelwi ? I see no problem adding the code, even if it's not relevant anymore.

I don't mind having this shortcut. @JackTemaki is this still relevant for you?

Note that the finished file is also put in the finished.tar.gz file at job cleanup.

Plan for review (discussed between @SimBe195 and @curufinwe ): We review and merge in 6 steps. All of the code can be reviewed but we will try to merge different...

This file is generated by the Makefile inside src/