-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Not sending all the data to FrSky Rx #5
Comments
Hey Pino I use the Mavlink 1 libraries. Mavlink 2 adds nothing the this solution. Oh, and I have "fastest" ticked in the Arduino IDE. From the log it seems that no Frsky telemetry is reaching the Taranis input pin, so it is likely not reaching the receiver in the air. If remember correctly you are using Air Mode together and a Pixhawk. Could you expand on the detail a bit? Pixhawk running APM PX4 firmware, or something else? I assume you know if you are running APM 3.5 or later, you can get S.Port Passthru telemetry straight out of the Pixhawk without this converter. Can you do a desk check with the solution again using some of the debug features ? |
Hello Eric.
Thx for the response.
Running 3.2.1 APM 2.6
Have to use Teensy.
I will use debug when I get back home late tonight. Don’t know why I didn’t just do that and send to ya. 😁
I originally did so and everything was working.
I will report back. Thx.
…Sent from my iPhone
On Jul 19, 2018, at 1:56 AM, Eric Stockenstrom ***@***.***> wrote:
Hey Pino
I use the Mavlink 1 libraries. Mavlink 2 adds nothing the this solution.
Oh, and I have "fastest" ticked in the Arduino IDE.
From the log it seems that no Frsky telemetry is reaching the Taranis input pin, so it is likely not reaching the receiver in the air. If remember correctly you are using Air Mode together and a Pixhawk. Could you expand on the detail a bit? Pixhawk running APM PX4 firmware, or something else? I assume you know if you are running APM 3.5 or later, you can get S.Port Passthru telemetry straight out of the Pixhawk without this converter.
Can you do a desk check with the solution again using some of the debug features ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Hello Eric, I have found what the problem is... What is the trick to have the Teensy poll or pull data from the APM to Initiate Data Stream? Many thanks... Pino. |
Hi Pino You must setup the SRx fields in Mission Planner, where x is the serial port you are using. This tells your APM flight controller to send out the appropriate telemetry types regularly. http://ardupilot.org/copter/docs/common-telemetry-port-setup-for-apm-px4-and-pixhawk.html If all else fails you can un-comment this line in the source, which will cause the Teensy to request the data streams every so often. Should you go this route, you must wire the Teensy tx pin to the APM rx pin (this connection is not needed otherwise). //#define Data_Streams_Enabled // Rather set SRn in Mission Planner Let me know how it goes. :) |
Good day Eric, X8R SPort S <–- TX1 Pin 1 |
Well, clearly i am getting the data. The Fuel, CURR, GAlt, seem to not be working... i will start to test them with debug logging. |
In yaapu`s LAU app you need to go into the menu, space down to voltage source and select the FC or your sensor |
Hi Eric, that option is enabled... ;-) |
Did you click on it and select the correct source? Yry the available sources to fix your low battery reading. |
Hello Again, the only source that gives data is FC. I connected my FrSky Voltage Sensor that connects to Balance Charger, did the discovery and then it added another sensor reading called CELLS. Now I see the correct voltage at 11.6v and 3.88v/cell under the VS option. What doesn't change is the GAlt, Hdop and the Fuel @99% or the mAh Consumed. The rest seems to be working... |
Hi togehter, |
Hello Markus, I will check if there is a new firmware for my X8R Rx. |
With the X8R RX everthing is fine. |
Ah, sorry for the misunderstanding. There seems to be one update for Horus feature on the X8R.. not relevant for me. ;-) I think there are some APM SR1 Settings that need to be tweaked... Here are some logs of testing with Debug Mode Enabled... #define Use_Local_Battery_mAh @5400 //#define Use_Local_Battery_mAh Strangely, GPS showing Lock but nothing else Detailed GPS Data and the Radio Log |
Good morning Gentlemen I see you had a busy weekend. R9: I'm afraid I know nothing about the R9 RX and will need to do some reading. It may or may not pass the 0x5000 range of telemetry. What you can do is un-comment //#define Frs_Debug_All and let's look at how the Teensy is interacting with the R9 firmware. Battery - volts, amps and mAh : From the log you posted it is clear that the output routine for 0x5003 Battery 1 is not receiving any data. This data comes from Mavlink message # 1 SYS_STATUS. So our next step is un-comment //#define Debug_Batteries. I expect you might then confirm that the record is arriving from your APM with zero in those fields. If that is so, then we need to determine why APM is not configured to send them. Have you GPS Status: Similar to above, we appear to not be receiving GPS data from APM. You can check if it is coming in by un-commenting these two lines: //#define Mav_Debug_GPS_Raw // #24 Heartbeat: ID #0 The heartbeat is very important for several reasons, one of which is it tells us what type of craft (or GCS or Tracker etc) is sending the mavlink telemetry. Flight modes and other things are determined according to ap_type. Please could you un-comment //#define Mav_Debug_Heartbeat Regards |
Good morning. I agree with all your assessments. I will test asap. |
p.s. #define Data_Streams_Enabled // Rather set SRn in Mission Planner Won't Compile...? |
Hi LTMNO The Srx settings work for everyone else, so I don't think you need to #define Data_Streams_Enabled. However, I will make sure it compiles in future in case. |
Hi again Markus If you want to patch , could you make this change: Note missing comma! WAS: MUST BE: const uint16_t mavRates[] = { 0x04, 0x0a, 0x04, 0x0a, 0x04, 0x04, 0x04}; I added in MAV_DATA_STREAM_EXTRA3 but missed the comma. :( I'm working with Alex on v0.05 of Plus, so I'll patch that one also |
Hello Eric, Alex send me the new version from his program witch is now working as a widget. He also wrote that he will add the transport from the senors to the Horus telemtrie. Cheers, |
Oops. Hi Markus. It's 23 C here today. Mid winter :) I meant to address the previous post to LTMNO. Sorry guys. |
Hello... Thanks for the reply... I will continue to play with the SRx values until I get a favourable result. Thanks again, Pino (LTMNO) |
Hi Pino I patched Plus v0.04 Try #define Data_Streams_Enabled again |
Hi Eric, Cheers, |
BHi Marcus Thanks for the 'heads up'. Plus is very much a work-in-progress I'm afraid. I re-jigged the options from v0.04 to support the Maple Mini. It's a lovely little STM32 board, affordable, boot-loader works properly through USB, and it has 3 additional serial ports. Anyway, please carefully check the options you have un-commented. Don't use auxilliary (bluetooth) option for now as I'm working on it. If you could check v04 again carefully I would appreciate the help. Thanks and regards |
Hi Eric, Cheers, |
Hello Eric, I tried to use the : "Plus v0.04 Try #define Data_Streams_Enabled again" version, it compiled for me but I was getting no data TELEMETRY on the radio side.. I am not sure what is the diff between the Plus and the v1.01 as I have been using the v1.0.1 this whole time. I have put the v1.0.1 back on and I am back to where we started.. i didn't have time today to test different Mhz rates in APM for SRx. I will tomorrow. |
Hey @LTMNO, can you please remove "@ 99" from your initial message? Not interested in receiving that many emails for the tread I don't really care. Thx. |
DART450-2018-08-20.xlsx |
Hello Markus Yes total mAh is sent back to the Taranis/Horus. I suggest that we trace the data through the system. You could un-comment in the Teensy
and monitor the output. Monitor for a while to see the mAh increasing over time. Once we know it is transmitted, we can look further. Regards |
Hello Again... okay, will do... I will report back shortly... |
Debug_Batteries.txt |
Ok Markus, the record type BATTERY_STATUS ( #147 ) is not coming in from Mavlink. I will post a quick alternative for you, where I accumulate mAh by addition of di/dt. i.e. sum of all the instantaneous current measurements over time. I'll look more closely tomorrow,. In the meantime please try v1.0.3b. Be sure to set all the #defines as per your requirements. |
Hey Thank you Eric... I will test now... Pino. ;-) |
Okay, had to fix this in the code to compile... was missing a comma
here is the output... Here is the screenshot of the Radio... Ah is showing up now and Fuel has changed... This is great news... |
Hello Eric, |
Yup, was me Pino aka LTMNO... I just didn't want to correct you... all good! ;-) |
Oops, very sorry Pino |
Pino, I would appreciate if you could give me feedback on its accuracy. The theory is sound, but when you add up an infinite number of small measurements, I'm not sure. We could tweak it if necessary. I'll also check up tomorrow to see why Mavlink did not pass down the APM measurement. |
All good. 😁
I will use my amp meter and measure tomorrow am. Thank you.
Pino
…Sent from my iPhone
On Aug 21, 2018, at 3:35 PM, Eric Stockenstrom ***@***.***> wrote:
Pino, I would appreciate if you could give me feedback on its accuracy. The theory is sound, but when you add up an infinite number of small measurements, I'm not sure. We could tweak it if necessary. I'll also check up tomorrow to see why Mavlink did not pass down the APM measurement.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.
|
It seems there was a short burst of activity on Battery Status #147, but then trickled out and I can't see where to activate it. |
Interesting…I am away on business… unfortunately, I can't test till tomorrow afternoon upon my return… sorry for the delay.
… On Aug 23, 2018, at 2:31 AM, Eric Stockenstrom ***@***.***> wrote:
It seems there was a short burst of activity on Battery Status #147, but then trickled out and I can't see where to activate it.
ArduPilot/ardupilot#3133 <ArduPilot/ardupilot#3133>
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub <#5 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEVRrkb8m2VCWAXp-EV43iZndjCciYvdks5uTkxfgaJpZM4VVTRt>.
|
Hi Pino Thank you for all this information. It helps fill in the picture, but the critical numbers for me are the average current during the 7 odd minutes that you did the test. If the average current was 3.591A over 7 minutes, then the mAh over that period = 3.591 * 7 / 60 *1000 = 419 mAh, which correlates with the energy you had to put back. So the 3.59A looks like it could be roughly correct, and the approximately 3000mAh on the Taranis can't be correct, especially after only a few minutes at 5A. Let me go back and do some more work. I want to look harder at how to get the APM mAh out of the FC remotely, second I'll do some current consumption tests of my own and see if I can calibrate the calculation with a correction-factor. Third, I want to look at the Mission Planner code. As Alex points out, everything is based on the current reading determined by the Pixhawk, in conjunction with the chosen power module, and its calibration. |
Hello Eric/Alex... Stay tuned... Thanks again for your feedback and help. Pino |
Hi Pino Please use the latest firmware on Github. Hope I'm not too late :) I did a lot of work on this today, and I believe it is fixed. I suspect the reason you are not getting the #147 record is because you are on an APM 2.6 FC, which only supports APM 3.2.1 and earlier firmware. So you have to use the di/dt method. The good news is that it now tracks very well with the one in Mission Planner (and APM 3.5.5). Go easy on your poor wife. :) Regards |
So please un-comment this line if you are not getting the #147 record type through
in this function
I'll make this automatic in future |
Hello Eric,
I then ran and didn't see message #147
then I un-commented
Question? I have done some serious attempts to calibrate my Power Module in APM... and it seems that based off of the Turnigy Amp Meter that I am off by 0.40 amps from the reading being sent down by FrSky. Perhaps putting a parameter in place to increase the value might be an easy fix? p.s. I am not sure if it is working at this point.. the counter.. i will have to charge again and start over and then record a flight to determine.. the increment is definitely slower... but the Fuel is not changing at all. |
Hello Again, I made a change in the debug to output this with the Battery Output.
With this I was able to record the starting and finishing point of the Amp meter output from a start and stop of consumption. now this is a short period of time... i will do a longer test tomorrow AM... |
Good sleuthing Pino! Yes you should try to emulate real-life conditions, higher currents and longer times. I have not built in a conversion factor, just increased float to double for the timing calc. So the method is still "pure", if you know what I mean? For you next FC consider getting a Pixhawk mini. They are now around the price of an APM 2.6 board, but support the latest APM firmware. The one catch is that the firmware (and Mission Planner) is modified by RadioLink (the vendor) first. |
Morning Eric. Yes sir. We are in business 😜👍
I believe to be whole now on the radio front.
Many thx!
…Sent from my iPhone
On Aug 28, 2018, at 2:32 AM, Eric Stockenstrom ***@***.***> wrote:
Your log reflects the numbers you mentioned above. That's as close as you need Pino! Is the same number appearing on your Taranis now?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Hello, Great solution... and I know the solution works as I had tried it a few days ago with the help of Yaapu and then today, I finally got around to doing two test flights and no data was coming thru... other than VFAS, Fuel @ 99% and RSSI and RxBt.
Here is the Radio Log...
Thoughts?
just remove the .xlsx from the file below to load.
DART450-2018-07-18.csv.xlsx
Question... when you compile in Arduino.. what settings did you choose?
I have read that Fast is the default....
Also, does your program use Mavlink1 or Mavlink2 or Agnostic?
The text was updated successfully, but these errors were encountered: