AirSim icon indicating copy to clipboard operation
AirSim copied to clipboard

Fix image saving in HelloDrone

Open webshared opened this issue 3 years ago • 3 comments

Fixes: #HelloDrone never asked to save image.

About

Fixed HelloDrone/main.cpp code to ask for path and save images. There was a simple bug.

How Has This Been Tested?

Run locally with AirSim, images saved as expected.

Screenshots (if appropriate):

After the fix applied:

 % ./main 
Waiting for connection - X

Connected!

Client Ver:1 (Min Req:1), Server Ver:1 (Min Req:1)
Press Enter to get FPV image
# of images received: 2
Enter path with ending separator to save images (leave empty for no save)
./img/
Image uint8 size: 62355
Image float size: 0
Image uint8 size: 0
Image float size: 36864
Press Enter to arm the drone

webshared avatar Aug 08 '22 23:08 webshared

Hey @atolkachiov, with the Open Source AirSim project being shutdown, no new features will be integrated. Would you like me to add this to a fork I created to continue development of the open source project? You can find it here: Colosseum (AirSim successor)

xxEoD2242 avatar Aug 09 '22 00:08 xxEoD2242

Hey @atolkachiov, with the Open Source AirSim project being shutdown, no new features will be integrated. Would you like me to add this to a fork I created to continue development of the open source project? You can find it here: Colosseum (AirSim successor)

Hey @xxEoD2242, sure pls merge into Colosseum, it has the same issue as I see.

webshared avatar Aug 09 '22 17:08 webshared

Hey @atolkachiov, with the Open Source AirSim project being shutdown, no new features will be integrated. Would you like me to add this to a fork I created to continue development of the open source project? You can find it here: Colosseum (AirSim successor)

Hey @xxEoD2242, sure pls merge into Colosseum, it has the same issue as I see.

Done -> Colosseum PR #2

xxEoD2242 avatar Aug 11 '22 09:08 xxEoD2242