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

fix_issues#364 #395

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

fix_issues#364 #395

wants to merge 4 commits into from

Conversation

neverbiasu
Copy link

当前,保存的配置文件名为纯数字时间戳,如 1709263480595.toml。这种命名方式容易混淆,难以区分不同的配置文件。

为了改善这一点,我对相关代码进行了修改,使保存的配置文件名包含 output_name 前缀,格式为 [output_name]_[timestamp].toml。例如,如果 output_name 为 "model",保存的配置文件名将变为 "model_1709263480595.toml"。

这种命名方式更加直观,便于识别和管理不同的配置文件。

修改详情

  • api.py 文件中,修改了toml_file的命名,确保下载的配置文件名也包含了 output_name 前缀。

修改了 #364

@Akegarasu
Copy link
Owner

output name may have invalid character for windows path.

@neverbiasu
Copy link
Author

I add a clean_filename function to remove the invalid character.🧹

mikazuki/app/api.py Show resolved Hide resolved
mikazuki/app/api.py Show resolved Hide resolved
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.

None yet

2 participants