-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
remove easy_tello dependency #9
Comments
I just noticed that easy_tello has archived their project, so we should remove this dependency if we get a chance. We should also link it to the DJI API somehow. |
Are you looking to connect it to the djitellopy python library instead of the easy_tello library? That would easily connect to the drone's command system. I believe I could get that connected pretty easily. |
We've been a bit caught up in some other development, so haven't looked into djitellopy yet. If you want to open a pull request with that change, we could do some testing with the drone we have and then merge it in. @sean-schaefer any other thoughts? Also, thanks so much for offering! I've continued to use this with some of the STEM outreach we do and I think it really helps students better understand the process compared to just sending commands to the drone... |
Yeah, I can definitely do that! I am happy to help. This simulator was actually super helpful as I am doing drone research for pathing and object avoidance and needed a python simulator for testing prior to real world testing. I should have it modified in the next week or so! |
djitellopy looks like it would be a great replacement. Thanks for your help @dullahgtt and let me know if you have any questions! |
So, I figured out all that needs to be done is replace the easytello library with the djitellopy library in the simulator.py file. All the components are the same from the looks of it and it runs perfectly. |
Given easy_tello is fairly simple, it might be better to just rewrite the API to the drone rather than use easy_tello.
The text was updated successfully, but these errors were encountered: