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

process: general traceability concept #343

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

masc2023
Copy link
Contributor

Document general concept for traceability of work products

Resolves: #319

@masc2023 masc2023 force-pushed the masc2023_create_general_concepts branch from c52dee6 to c70e771 Compare February 13, 2025 17:43
Copy link

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

@masc2023 masc2023 marked this pull request as ready for review February 13, 2025 17:48
Copy link
Contributor

Choose a reason for hiding this comment

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

In our architecture discussions we talked about linking component requirements to componenents and not subcomponents?
Same applies for the implementation. Also do we need the implementation as an integral part of our traceability concept? Or do we use the link to the implementation only for "user convenience"? Shall we also explicitly mention that implementation includes tests?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Implementation includes Detailed Design and Source Code, shall be part of our integral part of our traceability, compare ASPICE standard. Test are mentioned in addition, Unit Test or Component Integration Test.
Sub-components are part of a takeover, needs discussion, let's setup a special meeting to discuss that

Copy link
Contributor

Choose a reason for hiding this comment

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

Linking to components req: done in our pilot component (json reader) from component and from sub-component. And it made sense to do this as part of architecture work.

Copy link
Contributor

Choose a reason for hiding this comment

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

I assume we would also have test cases for feature requirements? So do we need to show how to link those to "implementation" as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We need to discuss feature and module definition again, seems not clear yet to everyone, also work products need re-visit, if you agree the we can add here the examples from POC here?

Copy link
Contributor

Choose a reason for hiding this comment

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

As I understand this view the feature integration tests are omitted in this picture on purpose. These are in wp_overview. In wp_overview "Component Integration Test" are in my opinion linked wrongly to "Implementation" because those test the integration of the sub-components to components, i.e. the Component Architecture and not the Detailed Design (part of Implementation). If there is no component architecture there also are no component integration tests. Maybe we need to show both situations?

Copy link
Contributor

Choose a reason for hiding this comment

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

Currently we aligned with the tool team that we would try to skip the tool requirements

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, can delete that

requirements level. Starting from top, the platform level, going down to feature, component
to the bottom the unit level. The concept is based on the classical V-Cylce approach.

.. figure:: _assets/score_traceability_model_wp_overview.svg
Copy link
Member

Choose a reason for hiding this comment

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

Any reason you are not using .drawio.svg as shown here: https://eclipse-score.github.io/score/main/guidance/docs-as-code.html#draw-io

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Was not part of discussion we used that already in the POC, why did you not consider that?

Copy link
Contributor

Choose a reason for hiding this comment

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

Does this docs-as-code guideline request to use VS code? Cannot another drawio app be used?

Copy link
Member

@AlexanderLanin AlexanderLanin Feb 19, 2025

Choose a reason for hiding this comment

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

Does this docs-as-code guideline request to use VS code?

That's not intentional. I'll have a look.

Cannot another drawio app be used?

Yes, but generally we want to have .drawio.svg files. I'm not sure whether those are technically different than .svg files, or whether that's purely a naming convention. We use .drawio.svg to clearly identify that those files can be edited via draw io.

Copy link
Member

Choose a reason for hiding this comment

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

Update: I've tried editing your .svg files. I can edit them. So it's purely a naming convention to store drawio-svgs as .drawio.svg.

Copy link
Member

Choose a reason for hiding this comment

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

Do we have sub-component requirements?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Under discussion as take-over from POC

Copy link
Contributor

Choose a reason for hiding this comment

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

No we do not have sub-component requirements, but these may be detailed enough to link to single sub-components

Copy link
Member

Choose a reason for hiding this comment

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

how about simple components which do not have sub-components?

Sorry about all the questions, maybe this is the wrong place.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Simple components are no problem, will add the examples images from the POC to make the definition more clear soon

Copy link
Contributor

Choose a reason for hiding this comment

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

as suggested above: maybe we add a traceability picture also for the components without sub-components.

Copy link
Member

Choose a reason for hiding this comment

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

I cannot find the difference between a component and a module anywhere in the documentation?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Need to take over examples images from the POC

Copy link
Contributor

Choose a reason for hiding this comment

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

this difference will be documented in detail in the architecture guidance, but maybe we need to spend a few words on the "modules" i.e. that they are the "bazel modules" containing the components?

Copy link

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

1 similar comment
Copy link

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

Document general concept for traceability of work products

Resolves: #319
@masc2023 masc2023 force-pushed the masc2023_create_general_concepts branch from af9459b to 2cce93e Compare February 16, 2025 09:56
Copy link

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

@masc2023 masc2023 self-assigned this Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

Improvement: Document General Traceability Concept for the workproducts
4 participants