AerospaceNU / AerospaceNU/better-engine-control-software

Make a global `ICommBoundary` variable for communication

Aperta
#14 0 commenti 0 reazioni 2 assegnatari Rivendicata da @ritika-gurjar Vedi su GitHub
enhancement help wanted question
Lingua principale
C++
Stelle
5
Fork
2
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.