search

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

reasoner.py and tomasys.py need to be refactored #158

Open Rezenders opened 1 year ago

Rezenders commented 1 year ago

Both reasoner.py and tomasys.py need to be refactored. It seems that some functions from tomasys.py should be in reasoner.py, and vice-versa. We need to define a clear separation of concerns, the line is a bit blurry.

For example, the obtain_best_function_design function from tomasys.py should be in reasoner.py since it is related to planning, which is something application specific.

chcorbato commented 1 year ago

This will probably be connected to #162 and #163