Updates type for LA metro sectors IDs (la_metro_sector)#209
Updates type for LA metro sectors IDs (la_metro_sector)#209jswiesner wants to merge 1 commit intoopencivicdata:masterfrom
Conversation
|
Hi @reginafcompton. It would be great to get your opinion about this proposed change. |
|
Hi @jswiesner – currently, I'm not involved in the OCD project. I'd suggest pinging Hannah Cushman, who supports LA Metro Councilmatic and its scrapers. Her insight will be more valuable than mine! |
|
Using a more generic Pinging @hancush to provide thoughts on this change. |
hancush
left a comment
There was a problem hiding this comment.
Seems reasonable to me, @jswiesner – thanks for the PR! As @jloutsenhizer suggested, please preserve the original identifiers by adding the revised identifier to the sameAs field, then adding the revised identifiers as new records. Relevant docs, if needed.
There were 4 IDs introduced in #139 which contain OCD ID types (
la_metro_sector) that are specific to one region, and therefore conceptually invalid. The OCD ID type should only provide the nature of the district, while the value should provide the specific location.I'd like to propose updating these IDs to instead use type
transit. These districts appear to refer to service areas of the LA county metro system (https://media.metro.net/about_us/service_sectors/images/map_service_councils.pdf), therefore the generic typetransitseems most suitable.