Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support backuping multiple single files #41

Merged
merged 1 commit into from
Nov 14, 2022

Conversation

liyc-ai
Copy link
Contributor

@liyc-ai liyc-ai commented Nov 13, 2022

This PR tries to support backuping multiple run files.

Currently, RLA supports backuping code dir or single run file. However, there exists the need to backup multiple single files. To support this, I modify the type of run_file from str to be str or List[str]. Right now, to backup multiple single files, we just need to wrap them in a list, like below:

rla_run_file = ['train.py', 'utils.py', 'draw.ipynb']

Then everything will be OK.

By the way, this PR also tackles a bug when the backup_code_dir in rla_config.yml is None.

@xionghuichen xionghuichen merged commit d005bd9 into xionghuichen:main Nov 14, 2022
@xionghuichen
Copy link
Owner

nice work~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants