Fix center vector length

microsoft / DroneRescue

This sample demonstrates how to bring AI to edge devices, by using AirSim to generate synthetic training data for a Custom Vision model and then how to deploy the model to edge devices.

MIT License

66 stars 25 forks source link

Fix center vector length #9

Closed laurelkeys closed 5 years ago

laurelkeys commented 5 years ago

Hi, I was reading through the drone orbitting code and realized that it seems like a pair of parenthesis is missing in the calculation of the center vector lenght.

I also saw that it's also missing it on the official AirSim repository (here). Should I send a PR there as well, or is this intentional?

msftclas commented 5 years ago

All CLA requirements met.

laurelkeys commented 5 years ago

Sorry, I'm re-opening this pull request here https://github.com/microsoft/DroneRescue/pull/10 since I'd made the changes on my fork's master branch.