designforcontext / aac_review_tool

MIT License
5 stars 7 forks source link

global Title mapping #48

Open VladimirAlexiev opened 7 years ago

VladimirAlexiev commented 7 years ago

Consider eg LOD CBMAA Titles.csv that includes eg this record:

ObjectID=     197
TitleID=      6,716  (someone printed the ID with a decimal separator, silly)
Title=        From "Album Unique" formerly in the collection of the Duke of Portland
TitleTypeID=  4
TitleType=    Group Title
DisplayOrder= 2
DataDate=     12/23/2015

The global Title mapping http://review.americanartcollaborative.org/entity/E22_Man-Made_Object#field_1-search_0 has the following problems:

workergnome commented 7 years ago
  1. We can't use crm:E78_Collection, because there's not an explicit collecting plan.
  2. Even if there were an AAT term, the museum has not reconciled their internal thesaurus to the AAT yet, so it's valid to use local thesauri.
  3. I don't see any issues using both SKOS and CRM together.
  4. It may have been the wrong decision, but I didn't think it was my responsibility (as the developer of the Browse app) to specify the URL patterns for the mappings. I can work with whatever, as long as they're conceptually valid. I agree that the mappings here are not valid, and will need to be fixed, but I didn't want to impose my preferences on how they should be assigned on ISI.
  5. I'm not going to touch the displayOrder issue. That conversation is out of my league.
azaroth42 commented 7 years ago

I agree that CRM is broken when it comes to ordering. That's a tradeoff of choosing CRM. displayOrder as a global property is also broken, it needs to be an overlay similar to ore:Proxy or schema:ListItem.

The issues with the definition of CRM's Collection are well understood and unsolvable.

Propose close.

VladimirAlexiev commented 7 years ago

@azaroth42 Should I post any issues at all then?

azaroth42 commented 7 years ago

As discussed, our target model is fixed for the current project. So any model changes are not in the scope of work for 2017 for AAC. Feel free to post issues, but expect not to get much traction on solutions.

VladimirAlexiev commented 7 years ago

All issues that I post in this project are for a future version. But you take them all as "propose to close" (not relevant? wrong? insurmountable?) So I'm wondering if I should bother to post any.

If we continue a specific discussion in this issue: CRM is not broken when it comes to ordering. CRM merely lacks ordering. The source data has a field DisplayOrder, which is often important, so it should be mapped. There are many ways to implement ordering: