CrayLabs / SmartSim

SmartSim Infrastructure Library.
BSD 2-Clause "Simplified" License
219 stars 36 forks source link

`DragonConnector` refactor #591

Open al-rigazzi opened 1 month ago

al-rigazzi commented 1 month ago

Description

The DragonConnector class should be refactored to ensure maintainability and readability.

Justification

The DragonConnector class contains at least one monolithic method, connect_to_dragon that is deeply nested and complex to read or maintain. Moreover, the class has too many responsibilities and should be split up in simpler, single-responsibility classes.

Implementation Strategy

The refactor should aim at reducing complexity, most likely by adding new single-responsibility objects, and splitting up nested and long methods.