Skip to content
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

Wrong frame size in documentation #213

Open
irudnyts opened this issue Dec 7, 2023 · 3 comments
Open

Wrong frame size in documentation #213

irudnyts opened this issue Dec 7, 2023 · 3 comments

Comments

@irudnyts
Copy link

irudnyts commented Dec 7, 2023

I noticed that the frame is actually 960x720, rather than 1080x720, as it is written here:

import pygame
from djitellopy import Tello

drone = Tello()
drone.connect()
drone.streamon()

frame_read = drone.get_frame_read()
print(frame_read.frame.shape)

drone.streamoff()

This is just typo, right?

@M4GNV5
Copy link
Collaborator

M4GNV5 commented Dec 8, 2023

Yes seems like a typo. Maybe the camera can do photos in 1080x720, but i guess we can still change the text to the correct format.
Fell free to send a pull request!

@irudnyts
Copy link
Author

irudnyts commented Dec 9, 2023

Will do. One more thing, I noticed that with different FPS I got different resolutions. I will try to prepare another reproducible example(s) and will get back with more details.

@irudnyts
Copy link
Author

irudnyts commented Dec 19, 2023

Here are my observations:

  • CAMERA_FORWARD:
    • LOW_RESOLUTION: (478, 648, 3)
    • HIGHT_RESOLUTION: (720, 960, 3)
  • CAMERA_DOWNWARD: (720, 320, 3)

Probably I should ask this in the other issue, but what exactly does the bottom camera frame consist of? If I understand correctly, the first 240 columns are camera pixels, but there rest seem to be related to infrared sensors. Is there any info on that? Once I get this info, I will send a PR.

Oh, and I was wrong, FPS does not affect the resolution, but FPS_5 does not work for me whatsoever.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants