search

sharedstreets / mobility-metrics

Tools for collecting, processing, and interpreting mobility data using SharedStreets
MIT License
50 stars 17 forks source link

Change MDS time unit #62

Closed jwoyame closed 5 years ago

jwoyame commented 5 years ago

Fixes #60

morganherlocker commented 5 years ago

This is good, but need to make sure providers are aligned before merging. I'll look into this today. In particular, I think Bird may be coming in as seconds, but I need to verify.

morganherlocker commented 5 years ago

I'm going to merge this. I'm simultaneously reworking providers to be generic, so we can notify providers that don't follow the spec.

thekaveman commented 5 years ago

Just a small note that in 0.2.x implementations, the unit of time is floating point Unix seconds.

It switched to integer Unix milliseconds in 0.3.0 (which incidentally also introduced the notion of version negotiation).

morganherlocker commented 5 years ago

Good catch @thekaveman. I'm working through the generic scraper this week, and will add the concept of per provider MDS version configuration there. This should allow us to normalize as needed during the caching process.