Tagging with date_format about greg

[manolomartinez]

Hi, thanks for your report. You were right and I was wrong. It should be

date_format = %%Y%%m%%d

to avoid interpolation errors, of course. Changing this in the config file, this seems to work in my tests. What is the problem you are seeing?

Manolo

[jstagge]

You are correct, this does work. My problem was the config setting:
tag_date = {date}
which throws the following error:
/usr/lib/python3.5/site-packages/stagger/tags.py:711: FrameWarning: TSOA: Ignoring incompatible frame
FrameWarning)
/usr/lib/python3.5/site-packages/stagger/tags.py:711: FrameWarning: TSOP: Ignoring incompatible frame
FrameWarning)
Traceback (most recent call last):
File "/usr/bin/greg", line 11, in
load_entry_point('Greg==0.4.7', 'console_scripts', 'greg')()
File "/usr/lib/python3.5/site-packages/greg/parser.py", line 138, in main
function(vars(args))
File "/usr/lib/python3.5/site-packages/greg/commands.py", line 176, in sync
downloaded = feed.download_entry(entry)
File "/usr/lib/python3.5/site-packages/greg/classes.py", line 331, in download_entry
aux.tag(placeholders)
File "/usr/lib/python3.5/site-packages/greg/aux_functions.py", line 198, in tag
stagger.util.set_frames(podpath, {tag: metadata})
File "/usr/lib/python3.5/site-packages/stagger/util.py", line 103, in set_frames
setattr(tag, key, value)
File "/usr/lib/python3.5/site-packages/stagger/tags.py", line 863, in date
self._validate_friendly_date(value)
File "/usr/lib/python3.5/site-packages/stagger/tags.py", line 450, in _validate_friendly_date
raise ValueError("date must be in 'YYYY-MM-DD HH:MM:SS' format")
ValueError: date must be in 'YYYY-MM-DD HH:MM:SS' format

So, this is a very minor issue caused by trying to tag with a modified date/time.

[manolomartinez]

Right, so the issue here is that, once you modify the date format, stagger complains, isn't it?
I can see how this is a nuisance.

One possible solution is to have a {date_stagger} placeholder that keeps the date in the format expected by stagger. It looks a bit clunky, perhaps. Other options?

[jstagge]

For my purposes, I am happy to not include the
tag_date = {date}

I think your solution makes the most sense. This is far from a critical item, but maybe could be introduced in the future.