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

Try compress video before send #442

Closed
progserega opened this issue Jul 28, 2019 · 9 comments
Closed

Try compress video before send #442

progserega opened this issue Jul 28, 2019 · 9 comments
Assignees

Comments

@progserega
Copy link

Users often try send big video to chat and it is success sending in other chats (such as whatspp, telegram), because other chats-clients repack video to small size on fly.
But not riot. Riot try send full video and often can not do that.

People be accustomed to this feature and want see it in RiotX.

@bmarty
Copy link
Member

bmarty commented Sep 24, 2019

Also it may lead to #587

@Tesla2k
Copy link

Tesla2k commented Jun 15, 2020

This is absolutely needed. No normal person will switch from WhatsApp or whatever to Riot if this is not added. (It should be added to all Matrix clients)

@Jouski
Copy link

Jouski commented Feb 7, 2021

Yes it would be a great idea to add this feature as in XMPP client Conversations.

@Fuzzillogic
Copy link

A participant in a group chat sent a video from their phone. The video was 92MiB. The user was on matrix.org, while other group members, as well as the group chat location, was on a self-hosted Synapse server. We could see the thumbnail, but trying to download or play the video resulted in nothing. While I still in the dark what this particular problem is (some room limits I'm not aware of?), it could have been avoided by recompressing the video to something much smaller, just like when sending photos. Most clips don't need the quality, and is wasteful when on mobile data bundles. The option to send the original is still welcome though; sometimes you do want the quality.

Also, VP9 WebM seems to work fine in Element. It would instantly improve compression rate over AVC, while maintaining quality. The same goes for using WebP instead of jpg and png. WebP also supports lossless, with compression much better than png.

@bmarty bmarty self-assigned this Apr 30, 2021
bmarty added a commit that referenced this issue Apr 30, 2021
bmarty added a commit that referenced this issue May 3, 2021
@bmarty
Copy link
Member

bmarty commented May 5, 2021

Implemented in 1.1.7 (#3264)

@bmarty bmarty closed this as completed May 5, 2021
@r4dh4l
Copy link

r4dh4l commented Jun 13, 2021

@bmarty Thank you for implementing this feature but from my experience the compression has almost no effect to the original size. Is there any way to set the compression level to what Handbrake does with Preset "General -> Very Fast 720p30" (with "Web Pptimized" enabled)?

@jtrees
Copy link

jtrees commented Jun 13, 2021

I can confirm that the file size for compressed files is always the same as the input file after "compression" for all videos I've sent so far (about 10).

@vetko
Copy link

vetko commented Oct 11, 2021

I can also confirm this, unfortunately: even when leaving the "Send video with the original size" unchecked, it does not change the size of videos (e.g. uploading a 200Mb video as 200Mb).
Have tried with multiple videos, different devices. The uploads are to encrypted rooms.

Should we open a new bug or will it be tracked inside this?

@progserega
Copy link
Author

Should we open a new bug or will it be tracked inside this?

I was open new issue about this bug: #4032

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

9 participants