self.armService = node.create_client(Arm, prefix + '/arm')

[ntukenmez3]

We would like to see the point in one command line in crazyflie.py. Are there any reason to toggle self.armService.wait_for_service() as comment?

self.armService = node.create_client(Arm, prefix + '/arm')
# self.armService.wait_for_service()

[knmcguire]

My guess here is that it is a blocking function where the rest of the script is blocked until the service is active, which is not handy in all cases. There was a while that the cflib backend didn't had this service I believe.

@whoenig is there a reason why its still commented? The cflib backend has an arming services now

crazyswarm2/crazyflie_py/crazyflie_py/crazyflie.py

Line 138 in b896e6d

# self.armService.wait_for_service()

[whoenig]

No real reason, we should add it. Technically it should only be needed to wait for the last service that is announced in the server, but might be "cleaner" to simply wait for all of them.