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

[Bug] (version .94) parameter -inp=ts not understood #1437

Closed
egossett opened this issue Jun 3, 2022 · 10 comments
Closed

[Bug] (version .94) parameter -inp=ts not understood #1437

egossett opened this issue Jun 3, 2022 · 10 comments

Comments

@egossett
Copy link

egossett commented Jun 3, 2022

Please prefix your issue with one of the following: [BUG], [PROPOSAL], [QUESTION].

I have a .ts file that contains programs 3, 4, and 6. I need subtitles for program 4.
I set the program number to 4 in "Obscure settings" and changed from "auto/default" to "ts" in "Basic settings"/input file format.

I get the error: parameter -inp=ts not understood

If I use "auto/default" for the input file format, the program number is ignored.

@egossett
Copy link
Author

egossett commented Jun 3, 2022

The parameter should be -in=ts

@canihavesomecoffee
Copy link
Member

That's a GUI issue (I believe you're using the GUI?), so @techno-disaster...

@egossett
Copy link
Author

egossett commented Jun 3, 2022

Yes, it is a GUI issue. Too bad the gui doesn't let you edit the command line parameters in the basic settings tab. Should be a 2-minute fix.

@techno-disaster
Copy link
Member

techno-disaster commented Jun 4, 2022

strange -in is -in according to https://github.com/CCExtractor/ccextractorfluttergui/blob/master/lib/models/settings_model.dart#L227

are you on the latest gui version? Also what do you see in the current commands box? Does that say -inp=ts as well?

@egossett
Copy link
Author

egossett commented Jun 4, 2022

I have the latest version. The command box shows "inp=ts". If I run CCExtractor from the command line with "inp" replaced by "in", it works. That means the GUI has a small error in creating the correct command line.

@techno-disaster
Copy link
Member

CCExtractor/ccextractorfluttergui@155596b should fix it :D

@egossett
Copy link
Author

egossett commented Jun 5, 2022

Thank you very much for the fix. Since I am running this on Windows 10 and not a Unix system, I will need to wait for the next non-source update to use this fix.

@canihavesomecoffee
Copy link
Member

Once the workflows finish you can normally download the artifacts for Windows, and replace the GUI exe in your installation folder. If you don't want to do that, yes, you'll have to wait until a new release is made.

@egossett
Copy link
Author

egossett commented Jun 5, 2022 via email

@cfsmp3
Copy link
Contributor

cfsmp3 commented Mar 22, 2023

Closing since this was fixed - @egossett free feel to reopen if you still have issues with the last GUI version.

@cfsmp3 cfsmp3 closed this as completed Mar 22, 2023
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

4 participants