<blockquote>
<p>There's obviously two major limitations with this code right now (the lack of the animated gif for the large picture, and the 1-sql-insert-per-tracepoint thing), but I'd rather see those fixed those in future PRs rather than doing everything in one PR.</p>
</blockquote>
<p>Animated gif support may be a bit tricky, as the repo owner hasn't been all that responsive lately. Maybe we'd have to fork it, and run our version of it, and find someone to finialize that animated gif part?</p>
<p>Bulk INSERT is quite crucial from a performance pov, otherwise this thing is barely usable. Given that we already have that code lying around (<a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="321490578" data-permission-text="Issue title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/1852" data-hovercard-type="issue" data-hovercard-url="/openstreetmap/openstreetmap-website/issues/1852/hovercard?comment_id=427116280&comment_type=issue_comment" href="https://github.com/openstreetmap/openstreetmap-website/issues/1852#issuecomment-427116280">#1852 (comment)</a>), and <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=147741" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/tomhughes">@tomhughes</a> already reviewed a few iterations, I don't see why we couldn't integrate this part right away.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2120#issuecomment-458110594">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLZ4Zu0630YH78yMoEIf0wNhqUcTdks5vHusggaJpZM4aDUsB">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABWnLUWWzZH4me8kHrcdSrirL0XwIZxuks5vHusggaJpZM4aDUsB.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/openstreetmap/openstreetmap-website","title":"openstreetmap/openstreetmap-website","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/openstreetmap/openstreetmap-website"}},"updates":{"snippets":[{"icon":"PERSON","message":"@mmd-osm in #2120: \u003e There's obviously two major limitations with this code right now (the lack of the animated gif for the large picture, and the 1-sql-insert-per-tracepoint thing), but I'd rather see those fixed those in future PRs rather than doing everything in one PR.\r\n\r\nAnimated gif support may be a bit tricky, as the repo owner hasn't been all that responsive lately. Maybe we'd have to fork it, and run our version of it, and find someone to finialize that animated gif part? \r\n\r\nBulk INSERT is quite crucial from a performance pov, otherwise this thing is barely usable. Given that we already have that code lying around (https://github.com/openstreetmap/openstreetmap-website/issues/1852#issuecomment-427116280), and @tomhughes already reviewed a few iterations, I don't see why we couldn't integrate this part right away."}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/2120#issuecomment-458110594"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/2120#issuecomment-458110594",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2120#issuecomment-458110594",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>