Skip to content

Conversation

@antonkri
Copy link
Contributor

@antonkri antonkri commented Oct 23, 2025

@github-actions
Copy link

⚠️ Docs-as-Code version mismatch detected
Please check the CI build logs for details and align the documentation version with the Bazel dependency.

@github-actions
Copy link

The created documentation from the pull request is available at: docu-html

Copy link
Contributor

@masc2023 masc2023 left a comment

Choose a reason for hiding this comment

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

Only comments, as author labeled it with needs-rework

# SPDX-License-Identifier: Apache-2.0
# *******************************************************************************
How-Tos
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this not called Handbook directly?

@@ -0,0 +1,16 @@

S-Core Book
Copy link
Contributor

Choose a reason for hiding this comment

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

in general we use in other documentation S-CORE (Big letters), the comment applies to the whole document

==========================

As it was already stated in the "Overview of the technologies" part, S-Core project consists of multiple
bazel modules, normally located in separate repositories. Majority of S-Core modules are part of S-Core
Copy link
Contributor

Choose a reason for hiding this comment

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

Bazel-Modules, according to our model Modules realized as Bazel-Modules


S-Core module is the central part of the s-core project, where the software architecture is defined. Here you will find the list and explanation of the features,
that are provided by the S-Core platform, definition of the high level architecture, break down of the high level architecture to the modules
and the definition of the functionality (logical interfaces) for every module.
Copy link
Contributor

Choose a reason for hiding this comment

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

You miss here PMP, which defines the deployment of the Process Definition in process_description, Tool Lists, Design Decisions, and Requirements

We automatically generate for every repository html documentation from rst files.
You can easily open it as shown at the picture (TODO: Link)

The process repository defines the S-Core process. It defines both general concepts and ideas of the S-Core software development process approach and
Copy link
Contributor

Choose a reason for hiding this comment

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

It defines Processes compliant to standards, a kind of Framework, Management System, independent from any tools. The PMP in the s-core repo documents, how they are deployed in S-CORE.
That's why it can also used for other projects outside S-CORE.

@antonkri antonkri force-pushed the ankr_score_book branch 7 times, most recently from 2018000 to 68b232e Compare October 30, 2025 14:25
@antonkri antonkri force-pushed the ankr_score_book branch 2 times, most recently from 6746b36 to dd73027 Compare October 30, 2025 14:53
@@ -0,0 +1,181 @@
<root> ([email protected])
Copy link
Member

Choose a reason for hiding this comment

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

remove file?

@antonkri antonkri force-pushed the ankr_score_book branch 2 times, most recently from 35f802b to 31290aa Compare October 30, 2025 16:35
--------------------------------------
For documenting our project, for specifying requirements, assumptions of use, architecture, detailed design, tests
and further software process related artifacts we completely rely on sphinx and sphinx-needs technology. Additionally we extend sphinx-needs
implementation with additional checks to ensure traciability and compliance to the S-Core metamodel. The S-Core metamodel & traceability concept
Copy link
Contributor

Choose a reason for hiding this comment

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

typo, traceability

allows to fix the problems directly on the implementation level and to ensure, that they will not happen in the future by always
reusing the same implementation.

- Another important aspect that normally comes up by the integration of the software stack, espeicially in ASIL relevant systems, is its quality
Copy link
Contributor

Choose a reason for hiding this comment

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

typo espeicially, especially, propose to replace ASIL with safety and security relevant

S-Core modules follow this process, mainly by automated checks and in case not yet implemented, with help of manual reviews. The clear
goal is to specify only those parts of software development process, that also can be validated with automated checks in the future.

It is important to understand, that S-Core project is not a final product, that can be "out-of-the-box" integrated into the automotive series projects.
Copy link
Contributor

Choose a reason for hiding this comment

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

propose to replace ASIL with safety and security

---------------
GitHub Link: https://github.com/eclipse-score

S-Core module is the central part of the S-Core project, where the software architecture is defined. Here you will find the list and explanation of the features,
Copy link
Contributor

Choose a reason for hiding this comment

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

and not to forget the objectives in form of stakeholder requirements and the AoU for the platforms for the potential users

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds comprehensive "How-Tos" documentation for the Eclipse S-CORE project, including a tutorial-style guide called the "Eclipse S-CORE Book". It also updates a Python dependency and modifies some configuration files.

  • Adds extensive documentation covering technology overview, architecture, module structure, integration process, and a sample application tutorial
  • Updates aspect_rules_py from version 1.4.0 to 1.6.3
  • Adds new lock files (uv.lock, dpex.txt) for dependency tracking

Reviewed Changes

Copilot reviewed 18 out of 29 changed files in this pull request and generated 13 comments.

Show a summary per file
File Description
docs/how_tos/index.rst Creates main index for How-Tos section
docs/how_tos/score_book/*.rst Adds comprehensive tutorial documentation including introduction, technology overview, architecture, module structure, and integration process
docs/how_tos/score_book/sample_application/*.rst Adds step-by-step tutorial for creating a sample S-CORE application
docs/index.rst Links the new how_tos section to main documentation
MODULE.bazel Updates aspect_rules_py dependency from 1.4.0 to 1.6.3
.vscode/settings.json Adds Python analysis path configuration
uv.lock, dpex.txt Adds new dependency lock files
.clwb/.idea/vcs.xml Updates IDE VCS configuration
Files not reviewed (1)
  • .clwb/.idea/vcs.xml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pipeline is done via bazel.
Validation of requirements, generation of documentation, building source code, execution of unit- or integration tests, for everything
there is a bazel target in Eclipse S-CORE. Not going into very technical details, bazel is a modern build system, that allows hermetical and
reproduciable builds, that are for special importance for building safety systems, and additionally provides the possibility to extend
Copy link

Copilot AI Nov 4, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'reproduciable' to 'reproducible'.

Suggested change
reproduciable builds, that are for special importance for building safety systems, and additionally provides the possibility to extend
reproducible builds, that are for special importance for building safety systems, and additionally provides the possibility to extend

Copilot uses AI. Check for mistakes.
multiple repositores, as you can see here (TODO: link to decision record). At the end, the decision was taken to proceed with multiple repositories.
The main reason for this, is that Eclipse S-CORE project should not only motivate software developers to work together on new solutions inside of Eclipse S-CORE
project but also empower to reuse as much as possible from already existing projects and this is easier to do with an approach, where every functionality
is encapsualted in its own repository. Such an approach has also disadvantages. First, the organizatorical approach of having so many teams working together.
Copy link

Copilot AI Nov 4, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'encapsualted' to 'encapsulated'.

Suggested change
is encapsualted in its own repository. Such an approach has also disadvantages. First, the organizatorical approach of having so many teams working together.
is encapsulated in its own repository. Such an approach has also disadvantages. First, the organizatorical approach of having so many teams working together.

Copilot uses AI. Check for mistakes.
multiple repositores, as you can see here (TODO: link to decision record). At the end, the decision was taken to proceed with multiple repositories.
The main reason for this, is that Eclipse S-CORE project should not only motivate software developers to work together on new solutions inside of Eclipse S-CORE
project but also empower to reuse as much as possible from already existing projects and this is easier to do with an approach, where every functionality
is encapsualted in its own repository. Such an approach has also disadvantages. First, the organizatorical approach of having so many teams working together.
Copy link

Copilot AI Nov 4, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'organizatorical' to 'organizational'.

Suggested change
is encapsualted in its own repository. Such an approach has also disadvantages. First, the organizatorical approach of having so many teams working together.
is encapsualted in its own repository. Such an approach has also disadvantages. First, the organizational approach of having so many teams working together.

Copilot uses AI. Check for mistakes.
let us now give it a try in the reference qnx image for the x86 platform.

Reference integration repository already provides a
`referance qnx image <https://github.com/eclipse-score/reference_integration/tree/main/qnx_qemu>`_.
Copy link

Copilot AI Nov 4, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'referance' to 'reference'.

Suggested change
`referance qnx image <https://github.com/eclipse-score/reference_integration/tree/main/qnx_qemu>`_.
`reference qnx image <https://github.com/eclipse-score/reference_integration/tree/main/qnx_qemu>`_.

Copilot uses AI. Check for mistakes.
Reference integration repository already provides a
`referance qnx image <https://github.com/eclipse-score/reference_integration/tree/main/qnx_qemu>`_.

The reference QNX image is currently really rundimentary, providing minimal set of tooling in the IFS image (TODO: link to QNX IFS image desc).
Copy link

Copilot AI Nov 4, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'rundimentary' to 'rudimentary'.

Suggested change
The reference QNX image is currently really rundimentary, providing minimal set of tooling in the IFS image (TODO: link to QNX IFS image desc).
The reference QNX image is currently really rudimentary, providing minimal set of tooling in the IFS image (TODO: link to QNX IFS image desc).

Copilot uses AI. Check for mistakes.
:align: center


Welcome to the Eclipse S-CORE project. As Eclipse S-CORE project is continiously developing, the decision was taken
Copy link

Copilot AI Nov 4, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'continiously' to 'continuously'.

Suggested change
Welcome to the Eclipse S-CORE project. As Eclipse S-CORE project is continiously developing, the decision was taken
Welcome to the Eclipse S-CORE project. As Eclipse S-CORE project is continuously developing, the decision was taken

Copilot uses AI. Check for mistakes.

It is important to understand, that Eclipse S-CORE project is not a final product, that can be "out-of-the-box" integrated into the automotive series projects.
This has multiple reasons. First of all, Eclipse S-CORE is an open-source project, meaning there is no any commercial company behind this project,
that can be liable for anything. The strength of the Eclipse S-CORE project is that both its software development process and the corrensponding tooling
Copy link

Copilot AI Nov 4, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'corrensponding' to 'corresponding'.

Suggested change
that can be liable for anything. The strength of the Eclipse S-CORE project is that both its software development process and the corrensponding tooling
that can be liable for anything. The strength of the Eclipse S-CORE project is that both its software development process and the corresponding tooling

Copilot uses AI. Check for mistakes.
--------------------------------------------------

We do encourage every software module inside of the Eclipse S-CORE GitHub organization or also outside of it
to follow the Eclipse S-CORE development process continiously and introduce it by including approtiated checks into
Copy link

Copilot AI Nov 4, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'approtiated' to 'appropriate'.

Suggested change
to follow the Eclipse S-CORE development process continiously and introduce it by including approtiated checks into
to follow the Eclipse S-CORE development process continiously and introduce it by including appropriate checks into

Copilot uses AI. Check for mistakes.
to follow the Eclipse S-CORE development process continiously and introduce it by including approtiated checks into
the CI/CD pipeline, so that compliance to the Eclipse S-CORE project is checked with every PR. But the fact is, that
we can not enforce this, even inside of the GitHub organization. This has various reasons. One reason could be,
that the software module already exists and is used in multiple other projecs. The other reason could be, that
Copy link

Copilot AI Nov 4, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'projecs' to 'projects'.

Suggested change
that the software module already exists and is used in multiple other projecs. The other reason could be, that
that the software module already exists and is used in multiple other projects. The other reason could be, that

Copilot uses AI. Check for mistakes.
]
],
"python.analysis.extraPaths": [
"./ubcode/lib/python3.12/site-packages"
Copy link

Copilot AI Nov 4, 2025

Choose a reason for hiding this comment

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

Path appears to contain a typo: 'ubcode' should likely be 'vscode' or '.venv'. This will likely cause the Python analysis path to fail.

Suggested change
"./ubcode/lib/python3.12/site-packages"
"./.venv/lib/python3.12/site-packages"

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants