Skip to content

Metamodel link renames#621

Open
aschemmel-tech wants to merge 1 commit into
mainfrom
aschemmel-tech-metamodel-link-renames
Open

Metamodel link renames#621
aschemmel-tech wants to merge 1 commit into
mainfrom
aschemmel-tech-metamodel-link-renames

Conversation

@aschemmel-tech
Copy link
Copy Markdown
Contributor

Relates: #418

@github-actions
Copy link
Copy Markdown

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

Copy link
Copy Markdown
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.

The linked ticket does not cover, why linke from component requirements is removed to the feature architecture, can you give a rationale for that?

@aschemmel-tech
Copy link
Copy Markdown
Contributor Author

The linked ticket does not cover, why linke from component requirements is removed to the feature architecture, can you give a rationale for that?

My understanding is that the metamodel picture shall reflect what is implemented (or to be implemented soon) in docs-as-code. This link is not implemented.
There is no mention of the usage of this link described in the requirements nor the architecture process.
When I thought of the correct naming of this link ("derived_from" I guess), I realized the above and removed this to reduce complexity of the pictures.

@aschemmel-tech aschemmel-tech force-pushed the aschemmel-tech-metamodel-link-renames branch from e2de5ad to 881bfc3 Compare March 26, 2026 10:55
@masc2023
Copy link
Copy Markdown
Contributor

The linked ticket does not cover, why linke from component requirements is removed to the feature architecture, can you give a rationale for that?

My understanding is that the metamodel picture shall reflect what is implemented (or to be implemented soon) in docs-as-code. This link is not implemented. There is no mention of the usage of this link described in the requirements nor the architecture process. When I thought of the correct naming of this link ("derived_from" I guess), I realized the above and removed this to reduce complexity of the pictures.

But it is marked as mandatory, so maybe it is forgotten? @RolandJentschETAS , can we discuss this, otherwise I am fine to remove it.

@RolandJentschETAS
Copy link
Copy Markdown
Contributor

The linked ticket does not cover, why linke from component requirements is removed to the feature architecture, can you give a rationale for that?

My understanding is that the metamodel picture shall reflect what is implemented (or to be implemented soon) in docs-as-code. This link is not implemented. There is no mention of the usage of this link described in the requirements nor the architecture process. When I thought of the correct naming of this link ("derived_from" I guess), I realized the above and removed this to reduce complexity of the pictures.

But it is marked as mandatory, so maybe it is forgotten? @RolandJentschETAS , can we discuss this, otherwise I am fine to remove it.

I would let that out. The lower level requirements shall consider the upper layer architecture, but this could also be done indirectly. comp_req -> feat -> feat_arc_sta, there is no direct link comp_req->feat_arc_sta necessary in my opinion.

@masc2023
Copy link
Copy Markdown
Contributor

@aschemmel-tech , ready to review and merge?

@aschemmel-tech
Copy link
Copy Markdown
Contributor Author

@aschemmel-tech , ready to review and merge?

no, have to rework to cover the "non-mandatory component architecture views" topic

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please consider #646

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please consider #646

@aschemmel-tech aschemmel-tech force-pushed the aschemmel-tech-metamodel-link-renames branch from 881bfc3 to 2088ec9 Compare April 10, 2026 14:34
@aschemmel-tech aschemmel-tech force-pushed the aschemmel-tech-metamodel-link-renames branch 2 times, most recently from 7de606a to 6190616 Compare May 12, 2026 12:45
@aschemmel-tech aschemmel-tech marked this pull request as ready for review May 12, 2026 12:52
@aschemmel-tech aschemmel-tech force-pushed the aschemmel-tech-metamodel-link-renames branch from 6190616 to 6a9be13 Compare May 12, 2026 12:55
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There is another building blocks image, you missed simplified version, on the bottom of the page

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

will update

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If I am correct, now fulfils is replaced by satisfies, but in the traceability and building block model is still fulfils, so unclear for me?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

no, will keep "fulfils" but change "belongs_to" to "satisfied_by" because we wanted to link the requirements to the architecture element (and not the view, which is also optional on component level).

:status: <draft|valid>
:tags: <process area or abbreviation>
:satisfies: <defined workflow:wf__<...>>, ..., <defined workflow:wf__<...>>
:derived_from: <defined workflow:wf__<...>>, ..., <defined workflow:wf__<...>>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

That implies, that you change it in the complete process model, thus you need also to change the process meta model so. Is that intended? Will the change done in another PR or in this PR?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

will remove this change

.. gd_req:: <Title>
:id: gd_req__<process>__<Title>
:satisfies: <link to guidance id>
:derived_from: <link to guidance id>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

`see same comment as above for process management, why do we have this twice? I think this here shall be deleted, only process management should manage process meta model elements, requiesments engineering is only for the building block model

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

will remove the duplicate here

@aschemmel-tech aschemmel-tech force-pushed the aschemmel-tech-metamodel-link-renames branch from 6a9be13 to 412eac8 Compare May 12, 2026 14:36
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.

3 participants