Skip to content

Conversation

@aschemmel-tech
Copy link
Contributor

Ref: closes #28

@github-actions
Copy link

github-actions bot commented Jun 6, 2025

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

Comment on lines +235 to +236
Based on the requirement versioning it shall be checked if a parent requirement was updated but not the linked child requirements.
In case an update was detected, the attribute requirement covered shall be set to "No"
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for clarifying this. 'Requirement covered' still is a bit weird sounding for this explanation but I think this makes it clear for us what to do. 👍

Copy link
Member

Choose a reason for hiding this comment

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

I'm not quite clear on the concept, as this sounds like we need to change the "requirement covered" automatically (and instantly) by modifying the rst files. But this is so far into the future, we don't need to clarify right now.

Copy link
Contributor

@MaximilianSoerenPollak MaximilianSoerenPollak left a comment

Choose a reason for hiding this comment

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

Makes sense from my side now.

Thanks for adapting.


Additionally there shall be the following logical groups of requirements:
* Assumption of use requirement
* Process requirement
Copy link
Member

Choose a reason for hiding this comment

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

"gd_req" is a process requirement? What about e.g. "wf__req"? Is "process requirement" an undefined group here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

wf__req should be a workflow of the requirements process, not a requirement. Process requirement = gd_req - Let's not mention standard requirements here as those are treated differently and I would say these are described in our process metamodel.

@aschemmel-tech aschemmel-tech force-pushed the aschemmel-tech-correct-req-process-req branch from 08bd11f to d1e4a3d Compare June 6, 2025 08:54
* For the remaining attributes only predefined values can be used. A more detailed description can be found here: :ref:`attributes of the requirements`
* Note that "rationale" is only mandatory for Stakeholder Requirements ...
* and process requirements do not need security and safety because these can be derived from the standards they comply to (as well type attributes as these would all be "Non-functional")

Copy link
Contributor

Choose a reason for hiding this comment

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

On that thought: Do feature requirements also need the security attribute? Or is the security determined from the feature architecture. This would mean that only component requirements would contain the attribute security?

:satisfies: wf__req__feat_req, wf__req__comp_req

Each requirement shall have a security relevance identifier:
Each requirement (apart from proccess req) shall have a security relevance identifier:
Copy link
Contributor

Choose a reason for hiding this comment

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

As mentioned above: do stakeholder and feature requirements also need the security attribute?

Copy link
Contributor

Choose a reason for hiding this comment

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

yes, why do you think, there is a difference from safety attribute?

:satisfies: wf__req__stkh_req, wf__req__feat_req, wf__req__comp_req

Each requirement shall have a automotive safety integrity level (ASIL) identifier:
Each requirement (apart from proccess req) shall have a automotive safety integrity level (ASIL) identifier:
Copy link
Contributor

Choose a reason for hiding this comment

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

Tool requirements?

AlexanderLanin
AlexanderLanin previously approved these changes Jun 10, 2025
@AlexanderLanin AlexanderLanin dismissed their stale review June 10, 2025 08:43

just noticed this is process :D

@AlexanderLanin
Copy link
Member

@hoe-jo @masc2023 can we merge this as-is and discuss further improvements elsewhere? This PR might not solve everything, but it is already an improvement as-is.

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.

Merge as it is now, continue for further improvements with another PR

@masc2023 masc2023 merged commit b6d46db into main Jun 10, 2025
8 checks passed
@masc2023 masc2023 deleted the aschemmel-tech-correct-req-process-req branch June 10, 2025 08:57
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.

Correct req mgt process req based on tool community feedback

6 participants