AerospaceNU / AerospaceNU/better-engine-control-software

Make a global `ICommBoundary` variable for communication

Ouverte
#14 0 commentaires 0 réactions 2 personnes assignées Réclamée par @ritika-gurjar Voir sur GitHub
enhancement help wanted question
Langage dominant
C++
Étoiles
5
Forks
2
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

In hindsight, I think it's best to prioritize the convenience of easy telemetry over cleanly modular design.

The core issue is that I think telemetry should be just as easy as printing to `std::cout`, just include and you can write your call. Currently, we have to pass a `ICommBoundary` through constructors, not ideal and requires non trivial refactoring and work.

I was thinking of making a basic interface like `bind_comm_object()` to set the global var, and for now `get_comm_object()` to get it. Later on I want to move to even more ergonomic interfaces for text logging and formatting.

If you want to try this issue, talk to me first. I'm not convinced this is a 100% good idea so I want to hear your opinion.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.