Skip to content
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

write ADR for Search by Location #269

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nziemecki
Copy link

Hi everyone! I'm Moki I'm working on my capstone for a Master's in GIS at Temple, and for my project I'm hoping to implement this seach by location feature into OpenDataPhilly. Lydia is my supervisor on the project, let me know if you have any feedback- this is my first ADR. I'm excited to be working on the website with you all!

@BryanQuigley
Copy link
Member

Welcome! Looking forward to your contribution!

I think taking a few datasets and seeing what data you'd parse would be useful to do with the ADR.

Assuming you are not parsing a datasets actual data, I feel the datasets are generally:

  • All of Philadelphia (all of the City of Phila category to my knowledge)
  • A bigger region (AKA DVRPC or PATCO)
  • State data
  • A dataset that's national or bigger (aka WalkScore

In those cases, it makes a lot of sense to just tag them in some way in the dataset definition (the work to do that can be postgres, etc), but I wouldn't expect it to change almost ever.

If you are hoping to parse all the datasets - I'm not sure how feasible that will be. Maybe make it opt-in on a per dataset basis.
Also see concerns from #264

@rcheetham
Copy link
Member

I like @BryanQuigley typology of city/county, regional, state, national/global and tagging this in the actual catalog entry and then limit your geo-enrichment process to city/county and regional data sets.

I would add to this by separating City/County datasets into a few categories as well:

I think you'll have to treat each of these differently and probably only address one type in an initial project. Obviously, the non-spatial data can't be done and imagery won't be meaningful. In addition, I think it will be very hard to develop the geographic summaries that you want the dynamic APIs, particularly the very large ones like property assessments and crime - it's not affordable to process them regularly and many of them are probably simply city-wide (in all geographic summary areas).

@lydiascarf
Copy link
Collaborator

I think we'll focus on this type of dataset type to start:

City datasets that are static (don't change much, unless they are re-published) and stored in ArcGIS - for example Air Monitoring Stations at https://opendataphilly.org/datasets/air-monitoring-stations/ and Land Use at https://opendataphilly.org/datasets/land-use/

We'll start by seeing what we can pull from a one-time analysis and put off handling updates. I think in general that each dataset's AOI is unlikely to change. We'll get a few datasets together and pull out the administrative boundaries as a proof of concept, then update this PR with our findings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants