-
Notifications
You must be signed in to change notification settings - Fork 14
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
[Requirements] 2. Get Image Close To #13
Comments
@cbeddow for the 5th option, 'date filter', what is the date format that the API accepts? I can't find date related filtering option on the APIv4 documentation, https://www.mapillary.com/developer/api-documentation/. Thanks! |
@Rubix982 the tiles will accept a filter based on Unix epoch milliseconds. https://www.unixtutorial.org/epoch-time/ So in the tiles, after requesting them we need to manually do maybe a filter like: min_date = 1547318486 The user might set a min_date and a max_date, or just one of those. My concern is that the unix epoch time is not user-friendly as an input. Most users on Mapillary search by date for filters, not often hours/minutes. I think we should have the input be in YYYY-MM-DD format (ISO 8601) as a standard: https://www.iso.org/iso-8601-date-and-time-format.html Example:
^ something like this. Or maybe the date range is a list, so we do:
And always require a min and max, or allow a wildcard type character like:
Not sure if Then if they want to be specific to hours and minutes, it is accepted to not just put YYYY-MM-DD but also to extend to YYY-MM-DDTHH:MM:SS. For example:
So we can read and accept both, but no partial format in between such as YYYY-MM-DDTHH, it must have MM:SS on the end, or have only YYYY-MM-DD if there is not a full THH:MM:SS. Then we convert that to epoch milliseconds and run a filter. How does that sound? |
Seems perfect and concise. Makes a lot of sense from the user's end. |
@cbeddow, for this part, And always require a min and max, or allow a wildcard type character like:
get_data(daterange=[2020-09-01,*])
Not sure if * is correct to use with a python function like this though. Does the |
@Rubix982 yes that makes sense, if unspecified means today. If the first argument is * or some wild card, then it should be epoch time of 0 |
@cbeddow The filtering we want to be available for this requirement needs to be joined via the 'https://tiles.mapillary.com/maps/vtp/mly_map_feature_point/2/{z}/{x}/{y}?access_token=XXX' # for id, value, first_seen_at, last_seen_at
'https://graph.mapillary.com/:image_id' # for fields, such as geometry and 'all' as mentioned above
'https://tiles.mapillary.com/maps/vtp/mly1_public/2/{z}/{x}/{y}?access_token=XXX' # For the organization_id to filter via organization key, and is_pano attribute for pano, flat or both The problem primarily is with the 2nd endpoint, passing any other field other than {
"error": {
"message": "Tried accessing nonexisting field (altitude) on node type (ChunkObject)",
"type": "MLYApiException",
"code": 100,
"fbtrace_id": "A514B-_ujHi2raqz20DgoGT"
}
} For now, I'll open up an issue to keep a track of this and reduce the default parameter from # From
'https://graph.mapillary.com/219262519730517/?fields=altitude,atomic_scale,camera_parameters,camera_type,captured_at,compass_angle,computed_altitude,computed_compass_angle,computed_geometry,computed_rotation,exif_orientation,geometry,height,thumb_256_url,thumb_1024_url,thumb_2048_url,merge_cc,mesh,quality_score,sequence,sfm_cluster,width&access_token=MLY|XXX
# To
'https://graph.mapillary.com/219262519730517/?fields=geometry&access_token=MLY|XXX |
@Rubix982 so the mly_map_feature_point should not be used in this. It should only call the mly1_public endpoint images layer, and retrieve the images within a radius of the input coordinates (lon,lat). It may be confusing that I was saying "point" which can mean a longitude,latitude, but also a map feature point which is anon-image, non-traffic sign piece of data such as a bench, crosswalk, street lamp or utility pole. But that should be the data the user is retrieving here. Let's review the code on this Thursday too. |
Is your feature request related to a problem? Please describe.
This issue deals with the 2nd requirement from the PRD over outputting getting "near image".
Describe the solution you'd like
NA
Describe alternatives you've considered
This feature has the following implementation needs,
Additional context
This makes an API call with the token set in R01 and returns a JSON object. It may make sense to set an option for the response format to be GeoJSON which slightly reshuffles the data format.
Reference:
The text was updated successfully, but these errors were encountered: