-
Notifications
You must be signed in to change notification settings - Fork 6
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
NPGObjThesTerms2 should be mapped #68
Comments
I mapped everything except for the shape and the sitter type. I was not sure the Shape goes in as P2_has_type. I felt the object 'has shape' Oval rather than 'is of type' Oval. But if you want me to map it that way, I'll go ahead and do it. |
@workergnome @azaroth42 what do you think? |
I don't think there's an easy way to explicitly model shape, so thinking of it as a classification rather than a description of geometry makes sense to me...it's not a great as it could be, but I don't know if there's a better middle ground within the CRM. |
👍 to P2. What we mean is "This object is an instance of the class of Oval things". |
NPGObjThesTerms2 includes very useful concepts that should be mapped:
I think that Sitter Type should be mapped to crm:P2_has_type such as "individual portrait", "group portrait", "non-portrait"...
The last col (URL Example) applies only if NPG does not map the respective set of concepts to AAT & TGN. If they map the concepts, use AAT directly. For TGN, use
tgn:nnnnn-place
not justtgn:nnnnn
The text was updated successfully, but these errors were encountered: