rename relations for SysML compatibility#618
rename relations for SysML compatibility#618RSingh1511 wants to merge 1 commit intoeclipse-score:mainfrom
Conversation
c6b1b9f to
b86cf94
Compare
masc2023
left a comment
There was a problem hiding this comment.
Not sure why you want to change it, but before you can change something, you need to apply change request, as this is defined here
https://eclipse-score.github.io/process_description/main/general_concepts/score_building_blocks_concept.html#building-blocks-meta-model
Thanks @masc2023 ! This is driven by issue #418 (Codebeamer import/export compatibility). |
This change of link names is planned by issue #418 since quite some time (also discussed and approved in the process community). |
It is laying around since a year, this is not planned, otherwise it should have been discussed in some round |
aschemmel-tech
left a comment
There was a problem hiding this comment.
See inline comments plus we need to update the "metamodel specification"
| :status: valid | ||
| :includes: logic_arc_int__example_feature__archex_logical_interface_1, logic_arc_int__example_feature__archex_logical_interface_2, logic_arc_int__example_feature__archex_logical_interface_3 | ||
| :fulfils: feat_req__example_feature__archdes_example_req | ||
| :derived_from: feat_req__example_feature__archdes_example_req |
There was a problem hiding this comment.
fulfils -> satisfies
| :safety: ASIL_B | ||
| :status: valid | ||
| :fulfils: feat_req__example_feature__archdes_example_req | ||
| :derived_from: feat_req__example_feature__archdes_example_req |
There was a problem hiding this comment.
fulfils -> satisfies
| :safety: ASIL_B | ||
| :status: valid | ||
| :fulfils: feat_req__example_feature__archdes_example_req | ||
| :derived_from: feat_req__example_feature__archdes_example_req |
There was a problem hiding this comment.
fulfils -> satisfies
| :safety: ASIL_B | ||
| :status: valid | ||
| :fulfils: feat_req__example_feature__archdes_example_req | ||
| :derived_from: feat_req__example_feature__archdes_example_req |
There was a problem hiding this comment.
fulfils -> satisfies
b86cf94 to
4fc7422
Compare
|
The created documentation from the pull request is available at: docu-html |
I have already updated the metamodel in this PR(eclipse-score/docs-as-code#446) |
Metamodel is here, is it in sync with that https://eclipse-score.github.io/process_description//main/general_concepts/score_building_blocks_concept.html ? Furthermore, this PR is not planned, no link to any issue, which milestone? etc. |
Created a "metamodel specification" update PR (#621) which also changes the wording in process_description tracker. |
aschemmel-tech
left a comment
There was a problem hiding this comment.
my comments fixed
4fc7422 to
34fe64b
Compare
34fe64b to
b4a304a
Compare
| :tags: prio_2_automation, general | ||
| :complies: std_req__iso26262__support_6433, std_req__iso26262__software_7414, std_req__iso26262__software_942 | ||
| :satisfies: wf__monitor_verify_requirements, wf__mr_vy_arch | ||
| :derived_from: wf__monitor_verify_requirements, wf__mr_vy_arch |
There was a problem hiding this comment.
Again you change something w/o changing the relevant meta model, https://eclipse-score.github.io/process_description/main/introduction/index.html
| Component Requirements | ||
| ---------------------- | ||
|
|
||
| .. code-block:: rst |
There was a problem hiding this comment.
using of directive for the development meta-model in process is not allowed, thus there are code_blocks, why removing them?
b4a304a to
6140539
Compare
6140539 to
3283cae
Compare
Again some concerns, the issues says compatibility to SysML, can we already target SysML 2.0, but would that not mean rather using "refine" as derived_from. |
Good point on SysML versioning. The naming follows SysML 1.x, not SysML 2.0 — as specified in issue #418. In SysML 1.x, «derive» / derived_from is the correct relation for requirements deriving from other requirements, and «satisfy» / satisfies is for architectural elements satisfying requirements. SysML 2.0's refine has different semantics and is out of scope for now, as current tooling (Codebeamer etc.) is based on SysML 1.x. |
masc2023
left a comment
There was a problem hiding this comment.
Still Process Meta Model is not changed. This contribution cannot be accepted. Please first do an official change request before you start working on breaking changes, which are not discussed and agreed in our process community
| module_name = "score_docs_as_code", | ||
| commit = "21640ab325b3aae147ba4e3e8b5e7ab89fc2e8f5", | ||
| remote = "https://github.com/etas-contrib/score_docs-as-code.git", | ||
| # TODO: revert to main once https://github.com/RSingh1511/docs-as-code/pull/new/rs/rename-relations-codebeamer is merged |
There was a problem hiding this comment.
Thanks for contribution, but it seems your change request are due to special tool requirements. This is not in scope with our concept. We discussed that already with @aschemmel-tech Please get in contact
There was a problem hiding this comment.
My understanding was that we are fine if SysML 2.0 use of the terms is still the same:
In my understanding "derived" relationship for requirements is still part of SysML 2.0: see https://www.omg.org/spec/SysML/2.0/Language/PDF section "9.6 Requirement Derivation Domain Library" or Figure 70 description. "Refinement" seems not to be requirement specific.
Also see https://www.omg.org/spec/SysML/2.0/Transformation/PDF : 7.8.8.3.1 DeriveReqt_Mapping "A SysML::Requirements::DeriveReqt relationship is mapped to a SysML v2 DerivationConnections::Derivation model library element."
"Satisfy" relationship also still part of SysML 2.0: See e.g. 7.21.1 Requirements Overview: "A design solution must satisfy the requirement and all of its member requirements and constraints to be a valid solution." or 7.21.4 Satisfy Requirement Usages
Also see https://www.omg.org/spec/SysML/2.0/Transformation/PDF : 7.8.8.3.22 Satisfy_Mapping "A SysML::Requirements::Satisfy relationship is mapped to a SysML v2 SatisfyRequirementUsage."
It is not a special tool requirement, it is just how we have configured our tooling (codebeamer and trlc) by using requirement attributes which conform to UML 1.x
|
|
||
| The first viewpoint is named as *feature architecture*. It displays the SW Components within the SW modules (= dependable elements) which are required to realize the feature including their interactions. Also the *logical interfaces* and the interaction between the feature and the user are included in this view. On this architectural level the feature requirements shall be allocated. A full rendered example for the static architecture is maintained in the | ||
| `module template documentation <https://eclipse-score.github.io/module_template/main/>`__. | ||
| The first viewpoint is named as *feature architecture*. It displays the SW Components within the SW modules (= dependable elements) which are required to realize the feature including their interactions. Also the *logical interfaces* and the interaction between the feature and the user are included in this view. On this architectural level the feature requirements shall be allocated. An example for the static architecture is shown here: |
There was a problem hiding this comment.
As previous comment, we do not use directives from the development meta model in process description, examples are now in module_template
There was a problem hiding this comment.
currently there is no example in the module_template, but I agree we should leave this untouched because it is already how it should be in the future.
As stated above a "metamodel specification" update PR (#621) was already created, but is stalled due to the "not mandatory component architecture" issue which impacts this. |
Meanwhile there are also changes to the process meta model, here, https://eclipse-score.github.io/process_description/main/introduction/index.html, this is not covered with that, and why should we to that, process requirements are not really derived |
Ah, sorry now I understand: it is about the PROCESS metamodel, i.e. the links from e.g. gd_req to workflows. This is not covered by the original issue and we should remove it from this PR. According to SysML 2.0 the relationship between gd_req to wf_ would be rather called "allocated to". We should discuss if we want this PROCESS model change in the process_community and then (if agreed) create a seperate PR. |
aschemmel-tech
left a comment
There was a problem hiding this comment.
Should remove the changes on gd_req links and also all other amendments to process description apart from linkages.
Summary
Renames SCORE relation attributes to align with Codebeamer conventions for future requirements/architecture import-export compatibility.
Related issue: #418
Changes
:satisfies::derived_from:feat_req,comp_req,gd_req, etc.):fulfils::satisfies:feat_arc_sta,comp_arc_sta,logic_arc_int, etc.)fulfils_backsatisfies_back