LACMTA / metro-api-v2

Docs: https://lacmta.github.io/metro-api-v2/ Dev: https://dev-metro-api-v2.ofhq3vd1r7une.us-west-2.cs.amazonlightsail.com/docs
https://api.metro.net/docs
0 stars 3 forks source link

mysterious map / title issue #539

Closed albertkun closed 3 months ago

albertkun commented 3 months ago

image

Issue

@anithegregorian

Cause?

Potentially a change in the API?

Debug attempts

image

anithegregorian commented 3 months ago

@albertkun the issue arose due to refactoring of shape_id to shape_ids (array), and removing JSON.parse() for geometries. For some routes it's working, quite a few are returning Internal Server Error, as you can see below:

image

albertkun commented 3 months ago

@anithegregorian ok! that makes sense! should i change it back to shape_id then?

edit: is the issue of many lines not showing up related to the current time being used? (some lines don't late at night) i notice that your time is set at 10pm, while it is current 10am here!

image

anithegregorian commented 3 months ago

That won't be an issue, my system time was 10:46PM (night), now it's using 24hrs format (morning in US)

I had to change these lines (not sure if change is required)

image

albertkun commented 3 months ago

ohhh, ok, gotcha! sorry about that! and thank you for the quick turn around!🙏 unless you have any other comments regarding this, i will mark this issue as resolved!

anithegregorian commented 3 months ago

@albertkun go ahead with closing the issue, I also noticed that the payload size is reduced drastically for geometries, a huge improvement! thx

albertkun commented 3 months ago

@anithegregorian no porblem! thank you for flagging the geojson issue! it really was a huge drop in payload size thanks to you! :)