-
Notifications
You must be signed in to change notification settings - Fork 15
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
scope 2.0 #100
Comments
Can we change scopeDescription to: "scopeDescription": [
{
"type": "dataset|attribute|feature|other",
"description": ""
}
] Type can be an enum, if necessary. Or we can leave as free-text with codelist and handle in the writers. |
The scopeDescription object has been changed as suggested. |
I also changed the roleExtent to timePeriod as per the discussion on issue #52. |
If we follow ISO the following would become our scope object. Now the we have it out-of-the-way of resource and associated with constraint, lineage, source, processStep, and maintenance scope becomes a means of limiting or constraining the block to a subset of the total resource. Note: scope is an array only in maintenance. We could probably get by with making this a single object. Let me know if you agree and are on board with changing scope to the following. {
"scope": [
{
"scopeCode": "MD_ScopeCode",
"scopeDescription": [
{
"dataset": "",
"attributes": "",
"features": "",
"other": ""
}
],
"sourceExtent": [
{
"see": "extent 2.0"
}
]
}
]
} |
I'm fine with this. |
One question. Does it make sense to have scopeDescription as an array? |
Not entirely, just being loyal to ISO. However with the array I can see the potential to group attributes and features to a dataset. |
What makes me question that is the use of plural attributes and features. |
Shouldn't that be scopeExtent? |
"scope" implements MD_Scope which is a new class used in many other classes throughout -1. Definition: "the target resource and physical extent for which information is reported".
Definitions:
Changes:
see ISO XML example useConstraint -3.xml
The text was updated successfully, but these errors were encountered: