ArduPilot / ardupilot

ArduPlane, ArduCopter, ArduRover, ArduSub source
http://ardupilot.org/
GNU General Public License v3.0
10.99k stars 17.53k forks source link

Feature suggestion: add vehicle name parameter #2182

Closed hsteinhaus closed 9 years ago

hsteinhaus commented 9 years ago

I would like to propose to add a vehicle name parameter, that makes it easier to remember the vehicle that a certain log belongs to.

It would also allow to auto-generate flight logbooks (as required in some legislations for UAVs) from dataflash logs via simple scripting.

magicrub commented 9 years ago

Like a param that is just a string of whatever you want? On Apr 27, 2015 11:11 PM, "Holger Steinhaus" notifications@github.com wrote:

I would like to propose to add a vehicle name parameter, that makes it easier to remember the vehicle that a certain log belongs to.

It would also allow to auto-generate flight logbooks (as required in some legislations for UAVs) from dataflash logs via simple scripting.

— Reply to this email directly or view it on GitHub https://github.com/diydrones/ardupilot/issues/2182.

magicrub commented 9 years ago

I like that, makes it much easier to identify logs from multiple aircraft. Very easy to do, I love the idea. On Apr 27, 2015 11:36 PM, "Tom Pittenger" tom@airphrame.com wrote:

Like a param that is just a string of whatever you want? On Apr 27, 2015 11:11 PM, "Holger Steinhaus" notifications@github.com wrote:

I would like to propose to add a vehicle name parameter, that makes it easier to remember the vehicle that a certain log belongs to.

It would also allow to auto-generate flight logbooks (as required in some legislations for UAVs) from dataflash logs via simple scripting.

— Reply to this email directly or view it on GitHub https://github.com/diydrones/ardupilot/issues/2182.

hsteinhaus commented 9 years ago

Exactly. Just a param to store a short alphanumeric label. I wonder that nobody else missed that yet.

magicrub commented 9 years ago

I'll jump on that, I'll submit a PR tomorrow for it. Easy to do. On Apr 27, 2015 11:39 PM, "Holger Steinhaus" notifications@github.com wrote:

Exactly. Just a param to store a short alphanumeric label. I wonder that nobody else missed that yet.

— Reply to this email directly or view it on GitHub https://github.com/diydrones/ardupilot/issues/2182#issuecomment-96946692 .

hsteinhaus commented 9 years ago

I think the main challenge is to find a nice central location (library?) that adds it to all vehicle types without touching three/four Parameters.pde files.

magicrub commented 9 years ago

That's my plan On Apr 27, 2015 11:42 PM, "Holger Steinhaus" notifications@github.com wrote:

I think the main challenge is to find a nice central location (library?) that adds it to all vehicle types without touching three/four Parameters.pde files.

— Reply to this email directly or view it on GitHub https://github.com/diydrones/ardupilot/issues/2182#issuecomment-96947146 .

rmackay9 commented 9 years ago

Duplicate so staying with tradition I will close the newer one. https://github.com/diydrones/ardupilot/issues/1411

AndKe commented 9 years ago

SYSID_THISMAV - does more or less just that ?

magicrub commented 9 years ago

Strings are best, just need to get them implemented On May 12, 2015 1:40 AM, "André Kjellstrup" notifications@github.com wrote:

SYSID_THISMAV - does more or less just that ?

— Reply to this email directly or view it on GitHub https://github.com/diydrones/ardupilot/issues/2182#issuecomment-101190408 .