-
Notifications
You must be signed in to change notification settings - Fork 0
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
API Schema for the service("people") controlling the State #1
Comments
I think the first version is better? Because I would convert the second one into the first one otherwise. |
I'm not sure if we want to carry extra data around in the same response. Per @MarcelCoding just a list of waypoints is easier for frontend, so I would just go with it. For anything else, if we need it, I would just move it to separate endpoint (since merging this into just a list of waypoints will be a pain). |
You mean don't send the coordinates directly? How should interpolation then be done? |
Nope, I only mean "extra features" mentioned above |
Ohh, I can't read, sorry xs idk a snapshot of the last data that is send over the websocket would be nice. |
@MarcelCoding U mean that we also would send the interpolated data over the websocket ? And the frontend would be reduced to pure display minimal logic ? |
Idk, at least that is how it is done currently. There would alps be other options. Idk, I think it does not realy matter how it is done. |
Yeah thats a different discussion I would say |
So I started building variant number 1. With the extra caveat that I will save in memory a history of time X. |
@revol-xut maybe worth looking at memcached? |
@oxapentane maybe worth for scalability but my worst case calculation would result of holding approx a 1GB in Memory Assuming we get approx 16 Waypoints per Second (currently we get like 2) But guess good point because assuming we have 20 Drivers using the app we will have waaaaaaayyyyy more data. |
Proposed API Schema
=> also working with waypoint
/vehicles/{region}
or more structured
Ideas of other features
The text was updated successfully, but these errors were encountered: