Albert Zeyer
Albert Zeyer
You mean to add this to the README? Feel free to make a PR. I would suggest to add it right before the "Alternatives / related links". > Might also...
For reference on the code, in `get_epoch_model`, this is the relevant case when training is done and `import_model_train_epoch1` is set: ```python elif config.value("task", "train") == "train" and import_model_train_epoch1 and start_epoch...
Please vote here on this comment, use: * 👍: if you think we should just change this without option (changing current behavior), or * 👎: if we should add an...
Ping? @curufinwe @michelwi @dthulke ?
Ok, so I guess it is Python 3.8 then. Or even newer? Where should this be put? CI run with this, so for the GitHub Actions configs. Maybe also README...
I think we also should wait for @curufinwe's feedback on this. He might know about some other setups.
@curufinwe ping?
Note: The min Python version influences also how Ruff formating behaves. I think e.g. the nice formatting for `with` is not possible with Python 3.8. (See https://github.com/rwth-i6/returnn/pull/1719.)
Same discussion to increase min Python version for RETURNN more: https://github.com/rwth-i6/returnn/issues/1720
One side effect I realized later: When some finished job never calls its `update` method, there are potentially some jobs missing in the graph. E.g. when you want to perform...