Improved method to get connections between models about brim HOT 4 CLOSED

I think that I'm going with the solution of adding a new class Connection(Requirement) and a non-abstract method define_connections. Connection will automatically have an is_submodel = False.

from brim.

I think that I'm going with the solution of adding a new class Connection(Requirement) and a non-abstract method define_connections. Connection will automatically have an is_submodel = False.

There is actually one big problem. Currently the choice is made to always first traverse the submodels as they do not define any connections with others at the same level. Adding connections does add this option making it possible that one tries to define something based on a relation, which does not yet exist. For example the wheel may try to compute its contact point using the ground without any relation on how the wheel is oriented w.r.t. the ground. This is not possible.

from brim.

While this could be fixed by adding an order attribute to the define functions. A problem that remains is that this entire feature is extremely sensitive to circular import errors.

from brim.

While this could be fixed by adding an order attribute to the define functions. A problem that remains is that this entire feature is extremely sensitive to circular import errors.

Option to solve it is to just say that it should be of type ModelBase in the case of connections.

from brim.