search

rodizio1 / EZ-WifiBroadcast

Affordable Digital HD Video Transmission made easy!
GNU General Public License v2.0
816 stars 200 forks source link

CPU temp and usage does not get forwarded for gnd pi #149

Closed Consti10 closed 3 years ago

Consti10 commented 5 years ago

Hello Rodizo, It seems like cpuload_gnd and temp_gnd does not get forwarded with the ez-wb forward struct. Can you have a look at it ?

Greetings, Constantin

bortek commented 5 years ago

I am trying to clean up the clutter @careyer , do you know if this is still relevant with 1.6RC6 image or can we close it ?

careyer commented 5 years ago

@bortek : Sorry I am missing the context to understand Constis question correctly. @Consti10 - can you give us a bit more context and details on your above question? We would be happy to address this problem if it still persists.

Consti10 commented 5 years ago

Version 1.6rc6 on both rpi's. FPV-VR connected via usb tethering. These are the values of the ez-wifibroadcast-forward-t struct: screenshot_2018-11-03-09-00-03

Note that cpuload_air and temp_air are transmitted, but cpuload_ground and temp_ground are not.

Hope this helps !

careyer commented 5 years ago

@Consti10 thank you for providing additional information. @rodizio1: Can you confirm that cpuload_gnd and temp_gnd does not get forwarded (or are empty) with the ez-wb forward struct? We could then apply the "bug" label

Consti10 commented 5 years ago

Hello everyone,

So I decided to go trough the EZ-WB code myself searching for the bug, and I noticed something strange: The only code related to the ez-wb status data forwarding is this file: https://github.com/rodizio1/EZ-WifiBroadcast/blob/26514f7d6d7f7ec42e3d16e98a4b164f6bd4870e/wifibroadcast/rssi_forward.c

but it seems to be the old one, forwarding the best rssi value. This makes me think: Is the code here on this branch even version 1.6 rc6 ? I mostly had to treat ez-wb as a big black working! box but since rodizio seems to be fully occupied by other means we might have to figure this one out ourselves

rodizio1 commented 5 years ago

Sorry for not replying earlier Consti. I'm back working on it. Currently, these values don't get forwarded, because it's just not implemented.

The Code on Github is not up-to-date, look into the 1.6.RC6 image for latest source. I'll see that I clean up this Github mess in the near future.