Currently how the run name is being generated like
resolved_run_name = run_name or f"{model}__{dataset}"
During project run the run directory is created like
def make_run_dir(output_config: Any) -> Path:
timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
run_name = output_config.run_name
run_dir = Path(output_config.root) / run_name / timestamp
run_dir.mkdir(parents=True, exist_ok=True)
return run_dir
So the dataset name can be string-reverse so the directory will be transformer__string-reverse, which looks weird and may fail.
I think we could use some alternatives
- Use generic name like run_name = "default_run" or "run1"
- Use project name itself as the nested run name.
- Normalize the model and the dataset name
Currently how the run name is being generated like
During project run the run directory is created like
So the dataset name can be
string-reverseso the directory will betransformer__string-reverse, which looks weird and may fail.I think we could use some alternatives