american-art / ima

Indianapolis Museum of Art
Other
6 stars 3 forks source link

Reference model for actor_type attribute in aac-actors.json #12

Closed mit2nil closed 7 years ago

mit2nil commented 7 years ago

Hi,

How do we model "actor_type" attribute in the context of actor objects in aac-actors.json. Actors are modeled as a crm:E39_Actor and there is not aboutness objects/properties describing type of the top level object unlike E22_Man-Made_Object object which is used for art works and not the artists.

Nilay Chheda

workergnome commented 7 years ago

@IllyaMoskvin (or whomever is the correct person from IMA to look at this), can you clarify how this type is used and what sorts of values it has? I don't know what "Registration-Verified" means, for example.

IllyaMoskvin commented 7 years ago

Sure thing. That field is pulled directly from our collections management system. Unfortunately, it appears to intermix workflow-related values with classifications. There's really only two actor types that are valid: "Artist" and "Organization".

To clarify: some of our objects have several actors associated with them. Typically, one of the actors is an artist of some type (e.g. "Artist", "Designer", "Painter"), and the other actors are organizations involved in the creation of the artwork (e.g. "Publishing House" or "Printer").

I'm pretty sure that we can write a script to deduce which of the two classifications each actor should have. Everything with a ULAN value should be an artist. Everything without is almost certainly an organization. Anything that has variations of designer, painter, and so on is an artist.

Are you guys differentiating between people and organizations in your mapping?

I have to go presently, but I'm working on a script to list all the unique values that field can take. I can get that your way, if it would be helpful.

P.S. Thank you for mentioning me. I was not watching this repository and did not notice the issues piling up. Will take a look at the backlog asap.

workergnome commented 7 years ago

We do have an explicit way to distinguish between people and organizations (E21 Person vs E40 Legal Body), so that would be helpful.

Probably worth spot-checking the ULAN to make sure—I know ULAN has a bunch of institutions in it, so unless you're explicitly using that as a rule at IMA, it might not hold true.

IllyaMoskvin commented 7 years ago

I can try to write a script with that logic, but it's been a couple days, and in light of the hackathon, I just want to clarify whether or not the students already wrote that code, just so that I don't cover the same tracks. Additionally, can you guys tell me what you'd like me to name the (new) field for each actor that will be used to indicate if it's an E21 Person or E40 Legal Body?

mit2nil commented 7 years ago

No, we haven't written any scripts for this so far.

SamiNorling commented 7 years ago

After reviewing the "actor_type" attribute and its values, it is almost exclusively used as a workflow/note field, to denote where information is coming from (ULAN, Registration-Verified), not what type of actor it is. The few instances where the value is "Person" is just messy data that should really indicate the source, and there is no corresponding identification of "Organization" or "Corporate Body" or anything similar in the data set.

So I would say that this field actually does not need to be modeled.

SamiNorling commented 7 years ago

Related to my previous comment about the actor_type attribute, should I remove this field from our data if it is not to be mapped? We do plan to refresh our data before the end of the summer, and I can include removal of this field as part of my clean-up if not mapping a field that exists in the data is a problem.

caknoblock commented 7 years ago

There is no need to remove any fields that are not being mapped. They will just be ignored.

On Jun 21, 2017, at 12:48 PM, Sami Norling notifications@github.com wrote:

Related to my previous comment about the actor_type attribute, should I remove this field from our data if it is not to be mapped? We do plan to refresh our data before the end of the summer, and I can include removal of this field as part of my clean-up if not mapping a field that exists in the data is a problem.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/american-art/ima/issues/12#issuecomment-310185867, or mute the thread https://github.com/notifications/unsubscribe-auth/ABB-qWLmjF96JfZG2LcAZMdMIKF7zCOOks5sGXOggaJpZM4LMWKz.

{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/american-art/ima","title":"american-art/ima","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/american-art/ima"}},"updates":{"snippets":[{"icon":"PERSON","message":"@SamiNorling in #12: Related to my previous comment about the actor_type attribute, should I remove this field from our data if it is not to be mapped? We do plan to refresh our data before the end of the summer, and I can include removal of this field as part of my clean-up if not mapping a field that exists in the data is a problem."}],"action":{"name":"View Issue","url":"https://github.com/american-art/ima/issues/12#issuecomment-310185867"}}}

SamiNorling commented 7 years ago

I suggest that this issue be closed.