Coder Social home page Coder Social logo

Comments (25)

webhoes avatar webhoes commented on June 29, 2024

following

from flightrecordparsinglib.

jeryini avatar jeryini commented on June 29, 2024

Would like to know this as well. Does it support new drones that are using SDK v5.x, such as Mavic 3 and M30? I'm running it on the logs from Mavic 3 and I'm getting the following exception:

./FRSample ~/Documents/DJIFlightRecord_2023-01-15_[17-38-14].txt
terminate called after throwing an instance of 'std::logic_error'
  what():  basic_string::_M_construct null not valid
Aborted (core dumped)

from flightrecordparsinglib.

WBawa avatar WBawa commented on June 29, 2024

@jeryini Out of curiosity, are both those drones fully updated with the latest firmware?

from flightrecordparsinglib.

jeryini avatar jeryini commented on June 29, 2024

@WBawa just updated the M3T with the latest firmware and did a test flight but I'm still getting the same error.

from flightrecordparsinglib.

webhoes avatar webhoes commented on June 29, 2024

I don't think the latest firmware should be an issue. If you have older logs like from last year, that was the latest firmware at that time. All those firmwares should also work today when using the library.

from flightrecordparsinglib.

jeryini avatar jeryini commented on June 29, 2024

Hello @webhoes

But see my error above. Would you like me to share the Flightlog txt file with you so that you can double check?

from flightrecordparsinglib.

WBawa avatar WBawa commented on June 29, 2024

@jeryini I'd appreciate it if you could send that to me too.

I was talking to DJI Support, and they suggested it may be a firmware issue, but it looks like it isn't.

from flightrecordparsinglib.

jeryini avatar jeryini commented on June 29, 2024

@WBawa here is the flight record using the DJI Pilot 2 app using latest firmware:
DJIFlightRecord_2023-03-20_[14-20-38].txt

Loading the file must be correct, as if I provide an invalid file to the script it returns load file failed instead of the above mentioned issue.

from flightrecordparsinglib.

webhoes avatar webhoes commented on June 29, 2024

from flightrecordparsinglib.

DJI-William avatar DJI-William commented on June 29, 2024

The file can be analyzed by our internal flight log tool, maybe there is a bug in this API.

from flightrecordparsinglib.

webhoes avatar webhoes commented on June 29, 2024

The file can be analyzed by our internal flight log tool, maybe there is a bug in this API.

That is the general question. Does this parser work with android created files. We can process IOS created files, but android created files report back as 'unsupported app' and other errors that are not documented.

Assuming you work for DJI, we would love to have more info about this or how to deal with these errors. Or have DJI fix them, that would be more awesome ofcourse.

from flightrecordparsinglib.

DJI-William avatar DJI-William commented on June 29, 2024

We will definately fix this issue. All app including offcial apps and MSDK apps/whatever iOS or Android should generate the same format FlightRecord.txt for all models.

from flightrecordparsinglib.

webhoes avatar webhoes commented on June 29, 2024

We will definately fix this issue. All app including offcial apps and MSDK apps/whatever iOS or Android should generate the same format FlightRecord.txt for all models.

That is great to hear! Any updates on a timeline? I sent Michael some detailed error screenshots. I can sent these to you too if you like.

from flightrecordparsinglib.

DJI-William avatar DJI-William commented on June 29, 2024

I can decode the flightRecord.txt using Mac environment, What environment you are using?
image

from flightrecordparsinglib.

WBawa avatar WBawa commented on June 29, 2024

@jeryini @DJI-William
I just figured out the issue, the generate.sh for Ubuntu is missing one of the build arguments that the Mac has. After adding it I got the same results as you William.

I will make a PR for this.

Result before:
image

Results after:
image

from flightrecordparsinglib.

WBawa avatar WBawa commented on June 29, 2024

@DJI-William just created the PR #4, if you could have your team have a look at that, that'd be great.

from flightrecordparsinglib.

DJI-William avatar DJI-William commented on June 29, 2024

Much appreciated to @WBawa. PR has been merged, you can close this ticket now.

from flightrecordparsinglib.

jeryini avatar jeryini commented on June 29, 2024

Good spot @WBawa ! Many thanks for the fix, I can confirm it now works. Though there is something else now. This is the output that I got for our app using MSDK:

{"summary": {
 "platform": "Android",
 "appVersion": [
  0,
  0,
  0
 ],
 "batteriesInformation": {
  "0": {
   "index": 0,
   "firmwareVersion": [],
   "serialNumber": "4ERPKBBEA153MG"
  }
 },
 "camerasInformation": {
  "0": {
   "index": 0,
   "firmwareVersion": [],
   "serialNumber": "1TCLK7K03BJ2JW"
  }
 },
 "gimbalsInformation": {},
 "remoteControllerInformation": {
  "index": 0,
  "firmwareVersion": [],
  "serialNumber": "5YSZKB30020TSG"
 },
 "flightControllerInformation": {
  "index": 0,
  "firmwareVersion": [],
  "serialNumber": "1581F5FJD22BF00B"
 },
 "aircraftName": "dji aircraft",
 "startTime": 1679318301.392,
 "startCoordinate": {
  "latitude": 46.253275637656635,
  "longitude": 15.096627885472678
 },
 "totalDistance": 92.8,
 "totalTime": 0,
 "samplingRate": 10,
 "maxHeight": 10,
 "maxHorizontalSpeed": 5.7706151,
 "maxVirticalSpeed": 3,
 "uuid": "8eadfe6d-4efe-4187-816e-fa598f85e775",
 "productType": 118
}
, "info": {
 "frameTimeStates": [
  {
   "gimbalsState": {},
   "camerasState": {},
   "batteriesState": {}
  }
 ]
}
}

Compared to log parsed from official DJI Pilot 2 app, it's missing the following data:

  • field firmwareVersion for batteriesInformation, camerasInformation, gimbalsInformation and flightControllerInformation;
  • remoteControllerInformation is missing
  • totalTime is always 0
  • frameTimeStates is empty array instead of data for each timestamp

Noticed even though log files are similar size, the parser output produces considerable less data for our app as witnessed above.

from flightrecordparsinglib.

jeryini avatar jeryini commented on June 29, 2024

Hello @DJI-William . Any new information on the above ^^ issue?

from flightrecordparsinglib.

DJI-William avatar DJI-William commented on June 29, 2024

Weird, is this always happened when MSDK generated the flight records file?

from flightrecordparsinglib.

WBawa avatar WBawa commented on June 29, 2024

Hey @DJI-William, would you also be able to take a look at these problem logs for us, they're both from the M300?

DJIFlightRecord_2022-11-14_[15-10-29].txt
DJIFlightRecord_2022-11-15_[11-02-17].txt

We see a decode failure when trying to decrypt these as seen below:
image

from flightrecordparsinglib.

DJI-William avatar DJI-William commented on June 29, 2024

We have no time to look at the bugs in Flight Record API currently, we are still working on MSDK now. We will come back later.

from flightrecordparsinglib.

jeryini avatar jeryini commented on June 29, 2024

Weird, is this always happened when MSDK generated the flight records file?

Yes, that is correct. Just tried a couple of flight and for all of them it's the same. Here are two additional flight logs for you two to check:
DJIFlightRecord_2023-03-31_[14-05-40].txt
DJIFlightRecord_2023-03-31_[14-20-18].txt

from flightrecordparsinglib.

jotatx avatar jotatx commented on June 29, 2024

How much you guys would charge to create a simple flight data parser to run in ubuntu ou win64?

because i fail to even download the docker image for the Flt Rec.

I am a researcher and it would reduce my current workload if i simply get a working parser.

cheers

from flightrecordparsinglib.

jeryini avatar jeryini commented on June 29, 2024

Hello @DJI-William . Any new information on the above ^^ issue?

Hello @DJI-William . Is there any updates on this issue?

Also the following:

The file can be analyzed by our internal flight log tool, maybe there is a bug in this API.

We will definately fix this issue. All app including offcial apps and MSDK apps/whatever iOS or Android should generate the same format FlightRecord.txt for all models.

So if I take the .txt flight logs from our app and put it into DJI Pilot 2 folder where it stores own .txt flight logs, they should appear in the DJI Pilot 2 Settings -> Flight record tool? The last time we tried this our logs were not shown in the DJI Pilot 2 flight record section. But we did see the compact logs under UAV Health management system -> Manage logs without having to copy them to the Pilot 2 app folder.

from flightrecordparsinglib.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.