-
Notifications
You must be signed in to change notification settings - Fork 8
[proposal] add disputed names #319
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
Conversation
2c54388
to
7cc9cbc
Compare
here's a formatted example to use for feedback
|
This change does not include the proposed schema support for disputed names: OvertureMaps/schema#319 One issue I discovered during this commit: LinkML's JSON Schema generator gets confused by indirection. I filed bug #2666 to the LinkML repo for this: linkml/linkml#2666
c12e42a
to
485bc45
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jonahadkins one thing I forgot if we discussed in all the back-and-forth: are we going to populate names perspectives with data for any features?
It would be good if the real life Gulf of Cartographer's Tears and Mount McTearly actually receive the names perspectives as a result of our data pipeline populating them correctly.
This is fully approved up. Merging. |
This commit applies the changes Jonah recently merged into the Overture schema in PR #319. OvertureMaps/schema#319 As modeled in this commit, all the perspectives models should be directly reusable for divisions when I get around to modeling that. (Which reminds me that one issue with Jonah's PR is that we reviewers totally failed to push for type reusability so there's duplication.) One revelation I had while modeling is that the way we do perspectives is essentially a form of scoping property. That's very interesting.
Category
What kind of change is this?
Please select one of the following five options.
Consult Pull request merging criteria for a description of each category.
Major change release plan
TODO: For any non-MAJOR change, delete this whole section.
For a MAJOR change as defined in Schema versioning and stability,
indicate the expected release date, related minor change steps, and your
public documentation and messaging plan.
A. Expected release date for this MAJOR change
TODO.
B. Related MINOR change steps
C. Public documentation and messaging lan
TODO.
Description
Perspectives for Geographic Features
We are all familiar with disputes and perspectives as a function of dealing with administrative names and borders. There also exist complex perspectives over geographic features. Common examples that our users may need to support are the Sea of Japan and the Falkland Islands.
This PR proposes the following changes:
is_disputed
to the base theme to flag features like mountain points, polygonal water bodies, polygonal land bodies, etc. that may contain disputed nameperspectives
to the base theme, when used in conjunction withis_disputed
this allows users to create the proper perspective view.disputed
as a valid value to the rules for names. Disputed names may be a separate thing from the common or official names.TODO.
Reference
List of relevant links to GitHub issues, PRs, and other documentation.
Testing
Brief description of the testing done for this change showing why you are confident it works as expected and does not introduce regressions. Provide sample output data where appropriate.
TODO.
Checklist
Checklist of tasks commonly-associated with schema pull requests. Please review the relevant checklists and ensure you do all the tasks that are required for the change you made.
A
but is not intended to test propertyA
's validity, and you made a schema change that invalidates propertyA
in that counterexample, fix the counterexample to align it with your schema change.Documentation Website
Update the hyperlink below to put the pull request number in.
[Docs preview for this PR.](https://dfhx9f55j8eg5.cloudfront.net/pr/<PUT THE PR # HERE>)