cityofaustin / atd-moped

A comprehensive mobility project tracking platform for Austin, Texas.
https://mobility.austin.gov/moped/
4 stars 3 forks source link

Add 4th layer to the Moped AGOL resource to house exploded multipoint geometry as points #1397

Closed frankhereford closed 1 month ago

frankhereford commented 2 months ago

Associated issues

https://github.com/cityofaustin/atd-data-tech/issues/17999

Testing

Thanks

Thanks to @mddilley for his help on this one -- brainstorming and rubber ducking. 🙏 🦆 🙏

Also thanks to @johnclary for his input and request to stick with our tried and true patterns. 💯


Ship list

frankhereford commented 2 months ago

@johnclary thanks for all your feedback! I am going to incorporate it as I rework this to use a view based on Mike and your direction here. I left you a note above with some more details about the choices and where it's going -- thanks again!

johnclary commented 2 months ago

ah—i see the separate PR now wrt the README. thanks!

frankhereford commented 2 months ago

@johnclary wrote:

it's my own (bad?) habit of expecting my docker container to have access to it's siblings. if we're going to remove this, would you also update the env_template with the correct local Hasura endpoint?

I updated environment template file -- thank you for that. 5b74d4a5311b54bef9852db45f49220cb7c923ac.

To your observation about using network=host or not; I guess my tendency is to try to lean into docker's ability to encapsulate and containerize, which makes me prefer a container not having access to the host's other services and being able to open ports on the host without explicit permission. Maybe this is a reflection of why I'm prone to write docker-compose.yaml files; i'm all about cementing that sibling relationship in configuration files, where you can have docker make one container genuinely dependent on another being available and all that. Your habit isn't any more wrong than mine; I think they are just different.

frankhereford commented 2 months ago

@johnclary wrote:

We have the option of slimming down the new exploded geometry view

Absolutely - great call. The view, query, and ETL have been refactored to do this.

mddilley commented 2 months ago

@frankhereford i merged the base branch (md-18877-update-proj-dev-status) into main but did not delete so i didn't pull the branch out from under you and if you prefer to manually re-target main