From: Pete Heist <notifications@github.com>
To: tohojo/flent <flent@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Subject: [Flent-users] [tohojo/flent] Stable output filenames for batch runs (#179)
Date: Fri, 06 Sep 2019 06:23:17 -0700 [thread overview]
Message-ID: <tohojo/flent/issues/179@github.com> (raw)
[-- Attachment #1.1: Type: text/plain, Size: 1049 bytes --]
Currently, the output filenames for batches always include `batch_time`:
```
def gen_filename(self, settings, batch, argset, rep):
filename = "batch-%s-%s-%s" % (
settings.BATCH_NAME,
batch['batch_time'],
batch.get('filename_extra', "%s-%s" % (argset, rep))
)
return clean_path(filename)
```
On one hand, this ensures that output filenames are unique, but on the other hand, one can't create stable links to the output filenames that don't change between runs, for example in a static HTML or markdown file.
What I could do is add a `filename_no_time` or similar parameter that if set to anything "truthy" (maybe if s in [ '1', 'true', 'yes' ]), excludes the time from the filename, then it's up to people to properly set `filename_extra`.
Would you be ok if I do that, or do you have any other suggestions for this?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/tohojo/flent/issues/179
[-- Attachment #1.2: Type: text/html, Size: 2390 bytes --]
[-- Attachment #2: Type: text/plain, Size: 151 bytes --]
_______________________________________________
Flent-users mailing list
Flent-users@flent.org
http://flent.org/mailman/listinfo/flent-users_flent.org
next reply other threads:[~2019-09-06 13:24 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-06 13:23 Pete Heist [this message]
2019-09-06 14:50 ` Dave Täht
2019-09-06 15:36 ` Pete Heist
2019-09-06 18:02 ` Toke Høiland-Jørgensen
2019-09-09 5:56 ` Pete Heist
2019-09-09 5:56 ` Pete Heist
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://lists.flent.org/postorius/lists/flent-users.flent.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=tohojo/flent/issues/179@github.com \
--to=notifications@github.com \
--cc=flent@noreply.github.com \
--cc=reply+AHVNJPZU2724FHDSMNEDDRV3P6PFLEVBNHHB2OOFUM@reply.github.com \
--cc=subscribed@noreply.github.com \
--subject='Re: [Flent-users] [tohojo/flent] Stable output filenames for batch runs (#179)' \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox