Skip to content
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

Change cardinality of Relationship "to" property from 1..* to 0..* #129

Closed
goneall opened this issue Mar 20, 2023 · 2 comments
Closed

Change cardinality of Relationship "to" property from 1..* to 0..* #129

goneall opened this issue Mar 20, 2023 · 2 comments
Milestone

Comments

@goneall
Copy link
Member

goneall commented Mar 20, 2023

If a Relationship has a completeness of noAssertion, there would not be an element for the to property.

This could be solved in one of three ways:

  1. Have a special Element NoAssertion that would be referenced in the to property. This is similar to today's solution.
  2. Change the cardinality of to from 1..* to 0..* and leave the completeness property as is (including a noAssertion value).
  3. Change the cardinality of to from 1..* to 0..* and remove the noAssertion value from the completeness property since having a to with no elements would imply noAssertion.

My preference would be 2 above is it is the cleanest to implement. The only downside I can think of is having a noAssertion value and having one or more to elements doesn't seem to make sense.

Having a NoAssertion element in solution 1 above is quite challenging to implement in some computer languages and in RDF, as we learned in the SPDX 2.X implementations.

@seabass-labrax
Copy link

seabass-labrax commented Mar 20, 2023 via email

kestewart added a commit that referenced this issue May 5, 2023
Update cardinality of "to" to have mincoun: 0 t as determined in #129
iamwillbar pushed a commit that referenced this issue May 5, 2023
Update cardinality of "to" to have mincoun: 0 t as determined in #129
@kestewart
Copy link
Contributor

Resolved in #293

@kestewart kestewart added this to the 3.0-rc milestone May 5, 2023
meretp pushed a commit to meretp/spdx-3-model that referenced this issue May 9, 2023
Update cardinality of "to" to have mincoun: 0 t as determined in spdx#129
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants