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

locn:address in the class foaf:Agent: the cardinality should be 0..* #107

Closed
jimjyang opened this issue Aug 15, 2022 · 5 comments
Closed

Comments

@jimjyang
Copy link

The cardinality for the property address (locn:address) in the class Agent (foaf:Agent) is now 0..1.

It should be 0..*, because an Agent may have several different addresses.

We are aware of that all properties in the class locn::Address have cardinality 0..*. However, when you for instance have postCode1 and postCode2, and postNameA and postNameB, how can you tell which one of the four combinations of postCode and postName is the correct one to use?

@jimjyang jimjyang changed the title locn:address in the class foaf:Agent: the cardinality should be 0..n locn:address in the class foaf:Agent: the cardinality should be 0..* Aug 15, 2022
@EmidioStani
Copy link
Member

Hello @jimjyang ,

enlarging a cardinality is not a big issue, I don't find a particular reason in the past. Do you have any practical example though ?

The properties of Address are optionals as they come from Core Location, so no constraint have been made within CPSV-AP.

@jimjyang
Copy link
Author

jimjyang commented Sep 9, 2022

@EmidioStani

Do you have any practical example though ?

The Norwegian Labour and Welfare Administration, the Norwegian Tax Administration etc., they all have their local offices in different cities, and several local offices in the same city, thus many different visiting addresses.

Even when you have one visiting address, you as a public organization usually have a poBox-address in addition, and the postCode and postName for your visiting address is usually not the same as the postCode and postName for your poBox-address.

The properties of Address are optionals as they come from Core Location, so no constraint have been made within CPSV-AP.

CPSV-AP does specify cardinalities for the properties in the class Address, which are all 0..*.

In our upcoming version of CPSV-AP-NO, we are considering to have

  1. 0..* for the property locn:address in the class Agent
  2. 0..* for all the xsd:langString-properties (free text, 0..* in order to support multilingual), and 0..1 for all the rdfs:Literal-properties ("coded value", max one value allowed/necessary).

@EmidioStani
Copy link
Member

Thanks @jimjyang , it is clear, it make sense that for this issue the solution will be the enlarging the cardinality.

The cardinality 0...* are shown because we simply drag and drop the Address class from Core Location showing as consequence all possible properties.

@williamverbeeck
Copy link
Contributor

During the webinar on the review of CPSV-AP on the 7th of November, it was agreed to relax the cardinality of relation Agent - Address (0..*).

@EmidioStani
Copy link
Member

This issue can be closed, it is solved in the new release: https://semiceu.github.io/CPSV-AP/releases/3.1.0/

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

No branches or pull requests

3 participants