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

"Error: Missing resolution" after running #1

Open
accuwau opened this issue Apr 19, 2024 · 9 comments
Open

"Error: Missing resolution" after running #1

accuwau opened this issue Apr 19, 2024 · 9 comments

Comments

@accuwau
Copy link

accuwau commented Apr 19, 2024

Heyo, not sure if you're still wanting to work on this, but when i run any command as such. I get the exact same error.

❯ ./nmb -s=580 -f=walkthrough.webm
Error: Missing resolution

any help is appreciated B)
I understand if you're no longer wanting to work on this though, I've abandoned quite a few projects myself

@djkato
Copy link
Owner

djkato commented Apr 19, 2024

I actually had fixed those issues months ago, just forgot to release them, sorry.
I just pushed new changes to crates.io, please update with cargo install! prebuilt binaries aren't released anymore.

@accuwau
Copy link
Author

accuwau commented Apr 24, 2024

haha no worries, thank you!

@djkato
Copy link
Owner

djkato commented Apr 24, 2024

@accuwau Lemme know if it fixed your issue :)

@accuwau
Copy link
Author

accuwau commented May 2, 2024

Aah apologies! just got chance to try it out again. it seems to work with most mp4 files but with webm or anything else i get this

❯ nmb -f test.webm
Error: missing duration

Adding -c WEBM doesnt help either sadly.

@djkato
Copy link
Owner

djkato commented May 3, 2024

Ah, you're right. I only tested mp4 cuz that's usually all I convert, but anything besides that breaks. I'll look into it soon.

@djkato
Copy link
Owner

djkato commented Jun 25, 2024

@accuwau looked at #2 , but I'm not happy with it, sorry. Apparently sometimes video streams are missing this data, but it can be extracted from metadata instead. I'll try to get a new version out that tries to substitute video data with its metadata.

@djkato
Copy link
Owner

djkato commented Jun 25, 2024

I solved the issue, but with latest ffmpeg it seems my app completely broke now 😢 ( #3 ). For now just use your own bugfix, sorry don't have time for this anymore

@djkato
Copy link
Owner

djkato commented Jun 25, 2024

pushed my changes, try if they work for you, or just use your own, rip on linux tho for me.

@accuwau
Copy link
Author

accuwau commented Jun 25, 2024

no worries haha, thought id just push my fix incase you wanted it. its definitely jank as ive never ever touched rust before. thanks for looking into it. i use your program a tonnn, worst case is ill just use mp4s 😁

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