On Tue, Nov 20, 2012 at 6:51 AM, Frederik Ramm <span dir="ltr"><<a href="mailto:frederik@remote.org" target="_blank">frederik@remote.org</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
(message to dev list but explicitly Cc'ing Brett and Scott because I don't know if they follow dev)<br>
<br>
   about a year ago, Marqqs tried to have a discussion on how to add timestamps to PBF files and hardly anyone was interested.<br>
<br>
I've had a couple people ask me whether I could somehow add timestamp information to the PBF files that I produce for <a href="http://download.geofabrik.de" target="_blank">download.geofabrik.de</a> so I'd be interested in solving this somehow.<br>
<br>
I really don't mind *how* it's done but I would really love to have one agreed way to place a timestamp in a PBF instead of everyone rolling their own.<br>
<br>
What's the current status of this discussion? Is there already an approved way to deal with this?<br></blockquote><div><br></div><div><div>No status, but if anyone wants my opinion, when authoring the format, I expected us to add metadata to planets, and expected it to be put into OSMHeader as in the OSRM clone you linked to above. I would vote to deprecate the use of the ISO timestring encoded into the optional_features array, but continue to write to it to avoid breaking old installs of Marqqs's tools. I also think that we have more than one notion of timestamp. How does this sound:</div>
</div><div><br></div><div><pre style="margin-top:0px;margin-bottom:0px;padding:0px;border:0px;font-size:12px;font-family:Consolas,'Liberation Mono',Courier,monospace;color:rgb(51,51,51);line-height:12.800000190734863px">
<div class="line" id="LC57" style="margin:0px;padding:0px 0px 0px 10px;border:0px">message HeaderBlock {</div><div class="line" id="LC58" style="margin:0px;padding:0px 0px 0px 10px;border:0px">  optional HeaderBBox bbox = 1;</div>
<div class="line" id="LC59" style="margin:0px;padding:0px 0px 0px 10px;border:0px">  /* Additional tags to aid in parsing this dataset */</div><div class="line" id="LC60" style="margin:0px;padding:0px 0px 0px 10px;border:0px">
  repeated string required_features = 4;</div><div class="line" id="LC61" style="margin:0px;padding:0px 0px 0px 10px;border:0px">  repeated string optional_features = 5;</div><div class="line" id="LC62" style="margin:0px;padding:0px 0px 0px 10px;border:0px">
<br></div><div class="line" id="LC63" style="margin:0px;padding:0px 0px 0px 10px;border:0px">  optional string writingprogram = 16; </div><div class="line" id="LC64" style="margin:0px;padding:0px 0px 0px 10px;border:0px">
  optional string source = 17; // From the bbox field</div><div class="line" id="LC65" style="margin:0px;padding:0px 0px 0px 10px;border:0px">  optional sint64 timestamp = 18; // Unix-Time encoded into varint of when the file was generated.</div>
<div class="line" id="LC65" style="margin:0px;padding:0px 0px 0px 10px;border:0px">  optional sint64 mirror_timestamp = 64; // Unit-Time timestamp of the last update the source. (used for mirroring)</div><div class="line" id="LC66" style="margin:0px;padding:0px 0px 0px 10px;border:0px">
}</div></pre></div><div><br></div><div>What about adding other metadata or adding in a nanosecond timestamp while we're at it?</div><div><br></div><div>Scott</div><div><br></div></div>