jonataslaw / jonataslaw/VideoCompress

VideoQuality enum "DefaultQuality" and "MediumQuality" have the same sizes

Open
#60 3 comments 5 reactions 0 assignees View on GitHub
Dominant language
Swift
Stars
262
Forks
344
PR merge metrics
No merged PRs in 30d

Description

First of all: Thanks for your good work! This is a lighweight package to video resizing. But I notice an elementar issue.
The enum `VideoQuality.HighestQuality` and `VideoQuality.HighestQuality` seems to output the absolutely same resolution. If I try to compress a video with my iPhone 11 and check the video width after compressing, I can see the following sizes:

`VideoQuality.LowQuality` : 224 Pixels
`VideoQuality.DefaultQuality` : 568 Pixels
`VideoQuality.MediumQuality` : 568 Pixels
`VideoQuality.HighestQuality` : 1280 Pixels

Why `DefaultQuality` and `MediumQuality` have the same sizes? Is this a known bug? I understand that more than 1280 pixels is not possible here because this are apple presets, but are this the real values regarding the video width of a landscape video?

Additional information: Before each try I called `await VideoCompress.deleteAllCache();` to be sure that there is nothing in cache.

My code:

```
for (int i = 0; i < videos.length; i++) {
var path = videos[i]["videoPath"];
final MediaInfo compressedVideo = await VideoCompress.compressVideo(path,
quality: VideoQuality.HighestQuality);
print(compressedVideo.width);
}
```

Thanks in advance for your information about this!

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the VideoQuality enum and the VideoCompress.compressVideo entry point described in the issue, then inspect how each quality maps to the reported output width. Reproduce the four quality measurements on an iPhone 11 after clearing the cache, and verify whether DefaultQuality and MediumQuality produce distinct documented resolutions.

Written by the indexing model from the issue text.

Assessment

Tech stack
dart, ios, swift
Domain
mobile-dev
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.