Skip to content

Conversation

@GsDublin
Copy link
Contributor

@GsDublin GsDublin commented Mar 1, 2022

png is to big! I got an error of sending 11 MB isnt supported.

png is to big! I got an error of sending 11 MB isnt supported.
Copy link
Owner

@kennybradley kennybradley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choosing jpg vs png should be an input argument. Maybe add that to the config file and check it before sending the image. Otherwise looks good, I didn't think about the maximum size because I use it on the "sub" stream which runs on 640x480.

@GsDublin
Copy link
Contributor Author

GsDublin commented Mar 5, 2022

I found the problem.
My streams are running on "main" channel because my version of reolinkapi didnt had the following implementation:
https://github.com/ReolinkCameraAPI/reolinkapipy/blob/5f3b6f9a5b4ee1fba7e362faea6174cd214b4506/reolinkapi/camera.py#L40

I dont know why it happend, i deleted the directory (/home/user/.local/lib/python3.8/site-packages/reolinkapi) and made a new clone.

Shall i made a new commit with a frame size/dimension check for the first frame of each cam?

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

Successfully merging this pull request may close these issues.

2 participants