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

Add 8 new models and updates #286

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Add 8 new models and updates #286

wants to merge 15 commits into from

Conversation

pedropaulofb
Copy link
Member

@pedropaulofb pedropaulofb commented Sep 2, 2024

Eight new models added. All models were sent via the catalog's Google Forms after we got in contact with the authors.

  • alkhalaf2024
  • barcelos2024resiliont
  • elghosh2024eucaim
  • nascimento2023spacecon
  • piest2024dsr-kb
  • pinheiro2024api
  • sariev2024maritime
  • schoonderbeek2024eamon

Model updated:

  • xhani2023xmlpo

All ttl files with the outdated purl.org Catalog URIs have been updated to the latest w3id.org URI.

@pedropaulofb pedropaulofb changed the title Add Resiliont Add 8 new models and updates Sep 4, 2024
context:
- research
source:
- https://tinyurl.com/5n8c7k8t
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should not use a GoogleDrive link with some images as the source. It should, instead, be the paper which includes these figures.

http://doi.org/10.5220/0012183900003598

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.
source substituted to https://dx.doi.org/10.5220/0000178400003598
landingPage removed.

representationStyle:
- ontouml
landingPage:
- https://hdl.handle.net/11311/1256092
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The landing page field should not point to a paper. The source field should be used for that.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.
source substituted to https://dx.doi.org/10.5220/0000178400003598
landingPage removed.

context:
- research
source:
- https://raw.githubusercontent.com/pedropaulofb/resiliont/main/resources/Ontological%20Foundations%20of%20Resilience.pdf
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace this link with the DOI of the paper.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no DOI yet. We can update the field once we have this piece of data.

representationStyle:
- ontouml
landingPage:
- https://catalogue.eucaim.cancerimage.eu/#/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does not seem to be the landing page of the ontology.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

URL removed.

context:
- research
source:
- https://link.springer.com/chapter/10.1007/978-3-031-47262-6_19
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the DOI instead.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

- enterprise architecture
- enterprise architecture management
- mining
- ontology
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ontology as a keyword is pointless.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

keyword removed.

context:
- research
source:
- https://link.springer.com/chapter/10.1007/978-3-031-64755-0_13
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the paper's DOI instead.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

representationStyle:
- ontouml
landingPage:
- https://essay.utwente.nl/101204/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not a landing page. Please remove.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

URL substituted by another that contains complementary information about the ontology.

context:
- research
source:
- http://essay.utwente.nl/101204/1/Sariev_BA_EEMCS.pdf
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should use the permanent url provided by the UT: https://purl.utwente.nl/essays/101204

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

representationStyle:
- ontouml
landingPage:
- https://link.springer.com/article/10.1007/s10270-023-01146-w
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not a landing page for the ontology.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

URL removed from landingPage. The paper does not mention a landingPage to the ontology.

Copy link
Member

@tgoprince tgoprince left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see my comments on the files.

@pedropaulofb
Copy link
Member Author

pedropaulofb commented Sep 10, 2024

@tgoprince, thank you for the review. All recommended adjustments have been made. Can you please review the PR again?

The modifications made were basically:

  • fix all purl.org URIs (except from DC terms)
  • fix reported landingPages and source pages
  • minor fixes resultant from the transformation to ttl

@pedropaulofb
Copy link
Member Author

@tgoprince , I fixed the mgic2011 dataset, which had non-OntoUML models. You can review it now.

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

Successfully merging this pull request may close these issues.

2 participants