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

All function designs in error are added to fd_error_log #141

Closed
3 tasks done
Rezenders opened this issue Nov 29, 2022 · 5 comments
Closed
3 tasks done

All function designs in error are added to fd_error_log #141

Rezenders opened this issue Nov 29, 2022 · 5 comments
Assignees
Labels
enhancement New feature or request ros2 ros2

Comments

@Rezenders
Copy link
Contributor

Rezenders commented Nov 29, 2022

Every time a function grounding violates a NFR the objective it is solving is added to the function design fd_error_log.
This shouldn't be the case for all violations, in the smart home for example we don't add the FDs to an error log. In addition, it doesn't make sense to block an FD for a violating a non-functional requirement.

Another issue I see is that the relationship order seems to be inverted. I would say that an FD is added to the objective error log and not the opposite.

But anyway, regarding the main issue I suggest:

  • Making a distinction between non-functional requirements and functional requirements
  • Only when a functional requirement is violated, the function design should be blocked

A quick solution for this in MROS could be to

@Rezenders Rezenders added enhancement New feature or request ros2 ros2 labels Nov 29, 2022
@Rezenders Rezenders self-assigned this Nov 29, 2022
@Rezenders
Copy link
Contributor Author

Done

@Rezenders
Copy link
Contributor Author

@chcorbato FYI

@chcorbato
Copy link
Contributor

We need to keep track of this change in the tomasys metamodel, i.e. values for 'Objective.status'
Have you edited tomasys.owl in some branch @Rezenders ?
Maybe good to indicate it here.

@Rezenders
Copy link
Contributor Author

Rezenders commented Dec 1, 2022

I only modified mros.owl and its SWRL rules. Changes are in the main branch. meta-control/mros_ontology@dd139f0
I didn't change the o_status. I will add that to tomasys.

@Rezenders Rezenders reopened this Dec 1, 2022
@Rezenders
Copy link
Contributor Author

Done at meta-control/mc_mdl_tomasys@a020191

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ros2 ros2
Projects
None yet
Development

No branches or pull requests

2 participants