meta-control / mc_mros_reasoner

library for metacontrol-based self-adaptation using ontological reasoning, with wrappers for robotic systems based on ROS1 and ROS2
Apache License 2.0
7 stars 10 forks source link

All function designs in error are added to fd_error_log #141

Closed Rezenders closed 1 year ago

Rezenders commented 1 year ago

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:

A quick solution for this in MROS could be to

Rezenders commented 1 year ago

Done

Rezenders commented 1 year ago

@chcorbato FYI

chcorbato commented 1 year ago

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 commented 1 year ago

I only modified mros.owl and its SWRL rules. Changes are in the main branch. https://github.com/meta-control/mros_ontology/commit/dd139f07854b52f3298d23c0a1232cce28cd538a I didn't change the o_status. I will add that to tomasys.

Rezenders commented 1 year ago

Done at https://github.com/meta-control/mc_mdl_tomasys/commit/a020191e7e7c52d4f5aa3e2240fca2ea269bf940