-
Notifications
You must be signed in to change notification settings - Fork 5
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
Tool broken after Mapillary update #11
Comments
Thanks for the ping. Unfortunately i don't have the bandwidth to patch this right now but am happy to update it if a patch is submitted. |
I had a go at updating it for API v4 https://tools-static.wmflabs.org/mapillary2commons2/ The main difference is that the username isn't provided by the API because Facebook has a blanket rule about this that they've been stuck with apparently. This version embeds the mapillary page with the username in it and has an input to type that in yourself so it gets attributed properly |
I should ask - would you prefer I maintain this separately, or would you like me to submit a pull request? I would like to keep tweaking it so if you wanted to keep it on this page and add me as a contributor that is an option as well. Or, I could just keep it on the separate github project/link. Whatever is easiest for you (assuming this works for everyone). |
Great work! I'm happy to add you as a contributor to this repo and a maintainer of the tool. Post a pull request and ping me your Wikimedia user name. I've updated the url used in the Mapillary-source template to use the v.4 api now. Because that template doesn't yet use the modern translation tool this change is only live for English for now. |
@bgo-eiu possibly a later patch but it should be possible to use something like beutifulsoup to extract the username (unless this violates some ToS) |
Alright, I can post a pull request. (Might not have time to get to it today, I intend to make some minor updates to the wording so that it doesn't say it's a fork anymore, and incorporate the accessible tagging suggestions that are recommended in the currently open pull request on this repository.) My username on the wikimedia toolforge system is middleriverexports (I assume that's the one you would need, they don't allow spaces on there so it's slightly different than Middle river exports which is what I have it set as on other sites). I was considering using something to scrape the username like that, but when I found out not exposing the username in the API was an intentional decision from Facebook I decided it might not be worth the trouble as I don't know if they would implement something to make scraping them more difficult (for example it used to be possible using the web GL end point to get the username, but they closed off this access point). One thing that could make things a bit smoother is to automatically fill in the username if it is already included in the URL. I noticed that for example if you start on an image in a sequence like this, you can click on the username and it gets added to the URL parameters: Which becomes: Then if you keep the username panel open and keep moving around in the sequence, the username remains part of the URL, so it could make things slightly smoother if you're in the habit of uploading images one after the other like this. |
Another thing that might be useful down the line is a way to add categories for the image (and structured data) quickly so that it doesn't end up uncategorized on Commons. Just now I removed the hidden category status from the "Images from Mapillary" category that the template uses, just because it can be helpful to find other Mapillary images without necessarily having hidden categories enabled (and potentially add more detailed categories to ones which don't have other categories). |
@bgo-eiu I've also merged the accessibility patch |
Hi there,
We made some platform updates recently and unfortunately they've broken this tool. A minor fix is required to get it working again.
The way image keys work has changed, and so a new URL needs to be used. Mapillary will then convert the old image keys to the new format.
The new url is in the following format:
https://www.mapillary.com/app/?&pKey=[ImageKey]
The text was updated successfully, but these errors were encountered: