Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Remove bib:Origin class #30

Open
rjyounes opened this issue May 8, 2017 · 9 comments
Open

Remove bib:Origin class #30

rjyounes opened this issue May 8, 2017 · 9 comments

Comments

@rjyounes
Copy link
Contributor

rjyounes commented May 8, 2017

Rationale: hasOrigin is similar to hasSource or subject: it's a relationship rather than a type of entity.

Involves three changes:

  • Remove bib:Origin class
  • Remove typing of origin individuals
  • Remove range specification on bib:hasOrigin.

Consider deprecating bib:Origin class rather than removing, for backward compatibility. But then in order to maintain full backward compatibility of all changes, need to deprecate bf:hasOrigin and define a new predicate without a range specification.

@rjyounes rjyounes changed the title Remove bib:Origin and range on bib:hasOrigin. Remove bib:Origin class and range specification on bib:hasOrigin. May 9, 2017
@rjyounes rjyounes changed the title Remove bib:Origin class and range specification on bib:hasOrigin. Remove bib:Origin class May 9, 2017
@sfolsom
Copy link

sfolsom commented Jun 16, 2017

Deprecation and a new property without the range specification sound good to me. Would the new property have the same name on a new ontology version URI?

@rjyounes
Copy link
Contributor Author

It wouldn't have to be a new property, would it? I was imagining removing the range from the existing property.

@rjyounes rjyounes added this to the Q2 2017 release (1.1.0) milestone Jun 16, 2017
@sfolsom
Copy link

sfolsom commented Jun 16, 2017

But is that good practice? idk. (probably doesn't matter in this stage of the model, but eventually we'll want to think about these type changes; implementors might be looking for these entailments)

@rjyounes
Copy link
Contributor Author

Is it good practice to remove the range, do you mean? That would be a non-backwards-compatible change, technically. I don't think we have to create a new term in order to change an existing term, do we? For this release, Jim and Simeon have proposed only a minor upgrade even though we have non-backward-compatible changes, because it seems odd (to me as well) to already release version 2. They thought it was acceptable because as yet we have no implementers, and we can add a note on the release stating that we're making an exception. In the long run we may need to modify the criteria, else I can see us being at version 4 or 5 pretty soon.

Originally this idea came from Christine - she thought Origin was like Source - that something is inherently an origin, but rather that something being the origin of something else is a relationship between two resources. That makes sense to me.

@eslao
Copy link

eslao commented Jun 16, 2017

I agree (surprise!) with what I said before, and I'm fine with removing the range from bib:hasOrigin and not creating a new class.

@rjyounes
Copy link
Contributor Author

Not creating a new property?

@eslao
Copy link

eslao commented Jun 16, 2017

@rjyounes ...Not creating a new property. Thanks.

@rjyounes
Copy link
Contributor Author

Adding to Q2 2017 milestone but it's fine if we defer it.

@rjyounes rjyounes removed this from the Q2 2017 release (1.1.0) milestone Jun 19, 2017
@rjyounes
Copy link
Contributor Author

Looking at this again, I don't find it clear-cut. First, hasOrigin already has an unspecified domain. This means that objects of the predicate may be independent resources, in which case the semantics of origin as a relationship between entities applies. Objects may also be named individuals of type bib:Origin, such as :addedTitlePage, :cover, etc. Note that these do not denote specific resources but rather (the beginnings of) a controlled vocabulary of origins. Where we want to create a binding resource (as ArtFrame and RareMat are proposing in certain cases) that could be the origin.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants