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 an overview section #121

Merged
merged 12 commits into from
Apr 12, 2021
Merged

Add an overview section #121

merged 12 commits into from
Apr 12, 2021

Conversation

create-issue-branch[bot]
Copy link
Contributor

closes #30

@kysrpex kysrpex linked an issue Apr 7, 2021 that may be closed by this pull request
@kysrpex kysrpex requested review from kysrpex and removed request for kysrpex April 7, 2021 12:46
@kysrpex kysrpex self-assigned this Apr 7, 2021
@kysrpex kysrpex marked this pull request as ready for review April 9, 2021 11:27
@kysrpex
Copy link
Contributor

kysrpex commented Apr 9, 2021

Visit http://127.0.0.1:8000/overview.html (Docker image) for reviewing. I did not include the new file on the TOC, yet, we might decide together where to put it. We could even merge it with the landing page as something that unfolds when clicking a button.

</figure>


Moreover, in SimPhoNy, all linked data is based on [ontologies](https://en.wikipedia.org/wiki/Ontology_(information_science)), which give **meaning** to the data. Specifically, SimPhoNy works with ontologies based on the [Web Ontology Language](https://en.wikipedia.org/wiki/Web_Ontology_Language), making the data compatible with the [Semantic Web](https://en.wikipedia.org/wiki/Semantic_Web).
Copy link
Member

Choose a reason for hiding this comment

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

We might later want to change this link for a reference to the explanation of ontologies in the introduction.

docs/source/overview.md Outdated Show resolved Hide resolved
docs/source/overview.md Outdated Show resolved Hide resolved
docs/source/overview.md Outdated Show resolved Hide resolved
docs/source/overview.md Outdated Show resolved Hide resolved
docs/source/overview.md Outdated Show resolved Hide resolved
docs/source/overview.md Show resolved Hide resolved
Copy link
Member

@pablo-de-andres pablo-de-andres left a comment

Choose a reason for hiding this comment

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

I would suggest moving here the requirement simplification and the coupling and linking from the old background section. I think it would fit here better than in the fundamentals

@kysrpex kysrpex requested a review from yoavnash April 12, 2021 07:21
@kysrpex
Copy link
Contributor

kysrpex commented Apr 12, 2021

I would suggest moving here the requirement simplification and the coupling and linking from the old background section. I think it would fit here better than in the fundamentals

Related to Yoav's comment. I'll continue the discussion there.

@kysrpex
Copy link
Contributor

kysrpex commented Apr 12, 2021

@yoavnash Do not forget to dismiss the change request if the solution is good enough.

@pablo-de-andres
Copy link
Member

What about the requirement simplification use case?

@kysrpex
Copy link
Contributor

kysrpex commented Apr 12, 2021

What about the requirement simplification use case?

Did not find it as simple neither had the time before the sprint review. 😉 I could do it this week, outside of the sprint framework.

@pablo-de-andres
Copy link
Member

ok, no problem. We should just keep it in mind, as it will be removed from the fundamentals section

@kysrpex
Copy link
Contributor

kysrpex commented Apr 12, 2021

ok, no problem. We should just keep it in mind, as it will be removed from the fundamentals section

Ok then I am opening an issue about this (not included in the sprint milestone)

@kysrpex kysrpex merged commit 19e4539 into dev Apr 12, 2021
@kysrpex kysrpex deleted the 30-Add_an_overview_section branch April 12, 2021 09:50
@kysrpex kysrpex added this to the Sprint #1 milestone Apr 12, 2021
@kysrpex
Copy link
Contributor

kysrpex commented Apr 13, 2021

ok, no problem. We should just keep it in mind, as it will be removed from the fundamentals section

Ok then I am opening an issue about this (not included in the sprint milestone)

Issue #142 opened.

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.

Add an overview section
4 participants