DJITelloPy icon indicating copy to clipboard operation
DJITelloPy copied to clipboard

Merge not included in pip build

Open NikEyX opened this issue 2 years ago • 1 comments

With so many people complaining about the video lagging, I was wondering why this library still uses OpenCV for capture. Then I saw in the commits that OpenCV HAS been replaced by PyAV in one of the PRs: https://github.com/damiafuentes/DJITelloPy/pull/120

And yet, the version on pip does not contain this change? It does, however, contain the chinese language change that came AFTER this PR???

Perhaps it would make sense to update the pip version to the current version on github?

NikEyX avatar Apr 21 '22 15:04 NikEyX

Ahh interesting, i didnt realise the pyAV parts are not released to pypi yet, i was already wondering because of issue #145.

To me it does not look like the chinese language changes are released yet either. Neither the readme on pypi nor the readme in the source distribution download includes the chinese version.

If i recall correctly i was afraid the pyAV change might cause more problems which have not been discovered yet, but i assume by now we can just release it to pypi.

M4GNV5 avatar Apr 21 '22 16:04 M4GNV5