-
Notifications
You must be signed in to change notification settings - Fork 74
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
Can H.download() add a new parameter which can help users define the download file names? #321
Comments
Note to self...It shouldn't be too hard to add an argument |
Thanks for the timely response.
I have just noticed there was a same issue.
Well, the optimization point should be something around the way of generating the filename. Now, it seems Herbie is using hash label as ordering tool, maybe we can use "target query time + subset name" as label, for full set, default subset name is "all".
The subset name is easy to generate, just extract only the numbers and alphabets from subset query name, about "target query time' we just extract only the numerical characters from query time.say, from subeset ":10[u|v]", we take only "10uv" for the subset name in downloaded file; from query time "2022-06-01 00:00", we take only "202206010000" as target query time.
Those labels should be enough to make files in order and meanwhile friendly for selection. Looks like there is no need to change any attributes of Herbie at the moment, maybe we don't need a new parameter, what we need is just a better file naming rule. :)
…------------------ 原始邮件 ------------------
发件人: "blaylockbk/Herbie" ***@***.***>;
发送时间: 2024年5月23日(星期四) 晚上11:05
***@***.***>;
***@***.******@***.***>;
主题: Re: [blaylockbk/Herbie] Can H.download() add a new parameter which can help users define the download file names? (Issue #321)
Note to self...It shouldn't be too hard to add an argument name to download that modifies the self.LOCALFILE attribute. Users could set name='my-name' or even use Herbie attributes name=f"{H.model}_{H.DATE:%Y%M%D%H}_my-name".
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
OK, "subset name" and "target query time" alone is not enough to order all subset files, there should be a "product" part also
…------------------ 原始邮件 ------------------
发件人: "blaylockbk/Herbie" ***@***.***>;
发送时间: 2024年5月23日(星期四) 晚上11:05
***@***.***>;
***@***.******@***.***>;
主题: Re: [blaylockbk/Herbie] Can H.download() add a new parameter which can help users define the download file names? (Issue #321)
Note to self...It shouldn't be too hard to add an argument name to download that modifies the self.LOCALFILE attribute. Users could set name='my-name' or even use Herbie attributes name=f"{H.model}_{H.DATE:%Y%M%D%H}_my-name".
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
the issue is as the title said.
The problems is when I download different subsets of the same model on the same date, it is hard to distinguish between them, manually renaming files can be a painful job. If H.download() can help solve this problem, herbie will just become more powerful.
Thanks for your nice work! Herbie has saved me a lot of time.
The text was updated successfully, but these errors were encountered: