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

hadProtocol should target nsg:Protocol objects #168

Open
wizmer opened this issue Aug 16, 2018 · 2 comments
Open

hadProtocol should target nsg:Protocol objects #168

wizmer opened this issue Aug 16, 2018 · 2 comments

Comments

@wizmer
Copy link
Contributor

wizmer commented Aug 16, 2018

In

the hadProtocol property targets "nsg:Entity". Shouldn't it target "nsg:Protocol" ?

@MFSY
Copy link
Contributor

MFSY commented Aug 24, 2018

Yes it should to be consistent.

@MFSY MFSY added this to the Neuroshapes v1 milestone Nov 27, 2018
@Ritikseptember2003
Copy link

@wizmer
You are correct that the nsg:hadProtocol property, based on its description - "Experimental protocols which produced traces having extracted features (protocols.json)" — should logically target "nsg:Protocol" rather than "nsg:Entity".

The reasons are:

  • Semantic Precision
  • Alignment with Description
  • Consistency

Some fixes also we can do like:

Replace
{
"path": "nsg:hadProtocol",
"description": "Experimental protocols which produced traces having extracted features(protocols.json)",
"class": "nsg:Entity",
"minCount": 1,
"maxCount": 1
}
with
{
"path": "nsg:hadProtocol",
"description": "Experimental protocols which produced traces having extracted features(protocols.json)",
"class": "nsg:Protocol",
"minCount": 1,
"maxCount": 1
}

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

3 participants