Skip to content
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

add (new) strava gps traces overlay #373

Open
wants to merge 4 commits into
base: gh-pages
Choose a base branch
from
Open

Conversation

@tyrasd tyrasd changed the title add strava gps traces overlay add (new) strava gps traces overlay Nov 8, 2017
@bhousel
Copy link
Member

bhousel commented Nov 8, 2017

I'd like to merge this as long as we can be confident that we really are allowed to use it in all of the OpenStreetMap editors for tracing.

👋 @drewrobb - we are big fans of the Strava heatmap!.. Can you let us know if there is a license that we can link the attribution text to? (see #323, #358 for some examples of what other companies have done).

@drewrobb
Copy link

drewrobb commented Nov 9, 2017

There are no plans to change this permission. I'm checking to see if a more explicit license could be provided for you all, but it may take some time.

@tyrasd
Copy link
Member Author

tyrasd commented Nov 12, 2017

Yeah, it's probably best for us to wait for a proper usage permission from @strava, especially given some ambiguous statements like openstreetmap/iD#2716 (comment) vs. https://twitter.com/paulmach/status/764125822441533441 in the past.

@bhousel bhousel mentioned this pull request Dec 5, 2017
@xsuchy
Copy link

xsuchy commented Dec 11, 2017

So did someone tried to contact Strava? I doubt that they will craft special license for us. But e.g., in Debian, it is perfectly fine, if we get a statement from someone@strava.com which state the same sentence as in https://wiki.openstreetmap.org/wiki/Strava#Data_Permission_-_Allowed_for_tracing.21 and when we save the email together with email's headers it can stand as a license.

@zdila
Copy link
Contributor

zdila commented Dec 11, 2017

FYI I've contacted Strava on Nov 23 to allow their heatmap on https://www.freemap.sk/. So far without any response. (We actually have it there already if expert mode is enabled but I plan to remove it.)

@xsuchy
Copy link

xsuchy commented Dec 11, 2017

@zdila which email you wrote to? Some general or someone personally?

@zdila
Copy link
Contributor

zdila commented Dec 11, 2017

@xsuchy maps@strava.com

@drewrobb
Copy link

Hi all, Good news: we have internally approved this usage, and are just trying to get the exact license/statement sorted out.

@Marc-marc-marc
Copy link
Collaborator

@drewrobb any news ?

@Spanholz
Copy link

Strava changed the layer to a pixelated version for lower zoom levels if you are not a Strava user yourself. Does this change anything?

},
"overlay": true,
"url": "http://heatmap-external-{switch:a,b,c}.strava.com/tiles/all/blue/{zoom}/{x}/{y}.png?px=256",
"max_zoom": 16,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems that max_zoom is now 12 after https://blog.strava.com/press/heatmap-updates/

Copy link

@Constable13 Constable13 Mar 15, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hi, looks to me like the heatmap now remains visible even when the zoom level is higher that 16, which is a really nice thing, but its quality decreased a lot, it's now pretty much useless to me.
How can we try fix this issue? Both Strava Slide and JOSM seem to have the same issue.

@naoliv
Copy link
Contributor

naoliv commented Mar 15, 2018

@tyrasd don't get mad at me, please.
There was an error from my part here :-/
The maximum zoom level is 11, although Strava gives a message saying "authentication is now required for data access beyond zoom level 12, please login to strava.com and then visit /auth"

For example, we can see https://heatmap-external-b.strava.com/tiles/both/bluered/11/750/1148.png but we can't see https://heatmap-external-c.strava.com/tiles/both/bluered/12/1505/2306.png

I am sorry for this mistake.

@tyrasd
Copy link
Member Author

tyrasd commented Mar 16, 2018

Ah, I believe zoom 12 is fine, because we're using the 256px version of the tiles: https://heatmap-external-c.strava.com/tiles/both/bluered/12/1505/2306.png?px=256

@grischard
Copy link
Collaborator

@drewrobb I'd love to get the heatmap into the osm editors, what's the mood like inside Strava? Anything we can do?

@grischard
Copy link
Collaborator

@drewrobb seems to have gone AWOL. Has anyone got contacts at Strava?

@doublah
Copy link

doublah commented Nov 25, 2019

@zdila
Copy link
Contributor

zdila commented Jul 16, 2020

BTW on www.freemap.sk we have Strava layer if you are logged in on Strava.

image

Otherwise you will see:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Strava heatmaps?