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

S Video Trim is worse than useless #861

Open
Rudxain opened this issue Aug 28, 2023 · 6 comments · May be fixed by #865
Open

S Video Trim is worse than useless #861

Rudxain opened this issue Aug 28, 2023 · 6 comments · May be fixed by #865

Comments

@Rudxain
Copy link
Contributor

Rudxain commented Aug 28, 2023

Your phone model: SM-A315G

Packages documentation to update:

com.samsung.app.newtrim

Documentation Change

com.samsung.app.newtrim

Removal: Advanced
➡️ Recommended

New name

Editor Lite

etc

This trimmer is both imprecise and inaccurate, ffmpeg is much better.

@AnonymousWP
Copy link
Contributor

It's not bloatware, but has a useful function, so I think Advanced fits better.

@AnonymousWP AnonymousWP linked a pull request Aug 29, 2023 that will close this issue
@Rudxain
Copy link
Contributor Author

Rudxain commented Aug 30, 2023

@AnonymousWP

It's not bloatware, but has a useful function, so I think Advanced fits better.

It would be useful, if it weren't for the fact that its only "feature" isn't done properly.

On my device (and, IIRC, also the SM-J327VPP) trimming from either end is inaccurate by multiple frames, especially the ending. Trimming from the last-end is inaccurate by 2 or more seconds, and it depends on the direction on which the "handle" was last slided by my finger (what?!)

This app literally had one job, and it does it poorly. It doesn't even deserve to exist.

The only reason not to use FFMPEG as an alternative, is because the app has a preview GUI, but even that is inaccurate...

It's like Samsung devs are underpaid, burnt-out, or just incompetent

@AnonymousWP
Copy link
Contributor

AnonymousWP commented Aug 30, 2023

Yeah I understand what you mean, but what if this is a temporary bug/issue and in the future it's working as intended? But yeah, if there's literally no reason to use S Video Trim instead of FFMPEG then I agree with setting it to Recommended.

And lol at the Samsung dev GitHub page, lmao.

@Rudxain
Copy link
Contributor Author

Rudxain commented Aug 30, 2023

@AnonymousWP

what if this is a temporary bug/issue and in the future it's working as intended?

I agree. It's possible they've fixed it in One UI v6.0, I'm on v4.1.

And lol at the Samsung dev GitHub page, lmao.

Thank you for reading the page, lol. It's getting kinda long, so I might split it in "category pages"

@AnonymousWP
Copy link
Contributor

@AnonymousWP

what if this is a temporary bug/issue and in the future it's working as intended?

I agree. It's possible they've fixed it in One UI v6.0, I'm on v4.1.

Could you try?

@Rudxain
Copy link
Contributor Author

Rudxain commented Sep 4, 2023

Could you try?

Samsung decided to stop sending updates to A31, so I can't test it... unless I try to install the newest video-trim from a 3rd-party source

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

Successfully merging a pull request may close this issue.

2 participants