AerospaceNU / AerospaceNU/better-engine-control-software

Make a global `ICommBoundary` variable for communication

Đang mở
#14 0 bình luận 0 reaction 2 người được giao Được @ritika-gurjar nhận Xem trên GitHub
enhancement help wanted question
Ngôn ngữ chính
C++
Star
5
Fork
2
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

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.

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.