-
-
Notifications
You must be signed in to change notification settings - Fork 313
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
Testing for the new release #1083
Comments
I no longer use subliminal or any other similar app because I no longer have a library of movies/series (my network server died 9 months ago and I won't be able to replace it), so I won't be able to help. Good luck to anyone who can :) |
I'm interested to test the new release ! We're using the subliminal python API to download subtitles in this project, and pip is used for subliminal installation. Btw, I followed the recent project handover, and I'm really happy that people are commiting to this projet again. |
Great! For now the updates are in the
Or you can wait for the next release. |
Hey, |
I can test too. |
You must already seen the results in the other threat but to summarize- When trying to refine with metadata
And when trying to use my key from omdb
My key seems to work fine in the test link that they send in the request respond email The rest of the logs seems pretty clean INFO and DEBUG of operations without problem. |
And while I was writing, a new pull to cover the issues. |
All the providers have been fixed, please test! Also to test, #1084 introduced a way to use a configuration file (in TOML format) with the CLI. It's not documented yet but there is an example file. |
This is still in the |
Ok, I made a subliminal.toml with my usernames and passwords as example one and placed it in ~/.config/subliminal/subliminal.toml. subliminal --debug download -p opensubtitlescom "The Matrix" omitted username and password, force, languages (es and en), etc because are all in the toml file, except TMDB (I don't have one) Error: Invalid value for '-r' / '--refiner': 'metadata' is not one of 'hash', 'omdb', 'tmdb', 'tvdb'. After removing metadata the configuration file seems to load just fine. subliminal --debug download -p opensubtitlescom "The Matrix" Using configuration file at /home/user/.config/subliminal/subliminal.toml Download both subtitles, all seems ok Same with addic7ed and the simpsons episodes. Trying with real files without specific provider. Some logging errors and sometimes guessing encoding. Attaching full log with an episode with both error types |
Thanks for testing and providing the logs, it's very useful! There are quite a few bugs, especially that Addic7ed and Gestdown are downloading corrupted subtitles for this episode.
…On 8 June 2024 02:04:50 GMT+01:00, Lovecraft-github ***@***.***> wrote:
Ok, I made a subliminal.toml with my usernames and passwords as example one and placed it in ~/.config/subliminal/subliminal.toml.
subliminal --debug download -p opensubtitlescom "The Matrix"
omitted username and password, force, languages (es and en), etc because are all in the toml file, except TMDB (I don't have one)
Error: Invalid value for '-r' / '--refiner': 'metadata' is not one of 'hash', 'omdb', 'tmdb', 'tvdb'.
After removing metadata the configuration file seems to load just fine.
subliminal --debug download -p opensubtitlescom "The Matrix"
Using configuration file at /home/user/.config/subliminal/subliminal.toml
Download both subtitles, all seems ok
Same with addic7ed and the simpsons episodes.
Trying with real files without specific provider.
Some logging errors and sometimes guessing encoding.
Attaching full log with an episode with both types
[full log.text.zip](https://github.com/user-attachments/files/15746005/full.log.text.zip)
--
Reply to this email directly or view it on GitHub:
#1083 (comment)
You are receiving this because you authored the thread.
Message ID: ***@***.***>
|
Tracked in #1126, you should see this error if you are using subliminal with python 3.12. A quick workaround for now is to manually install |
all the rest is solved hopefully.
I don't use discord so I would personally prefer a Matrix room. |
How can I do that? I tried pip -U, and it said that I need to use pipx or pacman. |
If you installed subliminal with `pipx`, you can try `pipx inject subliminal setuptools` to install `setuptools` in the subliminal venv.
…On 18 June 2024 04:50:26 GMT+01:00, Lovecraft-github ***@***.***> wrote:
> > Error: Invalid value for '-r' / '--refiner': 'metadata' is not one of 'hash', 'omdb', 'tmdb', 'tvdb'.
>
> Tracked in #1126, you should see this error if you are using subliminal with python 3.12. A quick workaround for now is to manually install `setuptools` in `subliminal` venv.
How can I do that?
I tried pip -U, and it said that I need to use pipx or pacman.
pacman has python-setuptools already installed and pipx said "No apps associated with package setuptools or its dependencies. If you are attempting to install a library, pipx should not
be used. Consider using pip or a similar tool instead."
--
Reply to this email directly or view it on GitHub:
#1083 (comment)
You are receiving this because you authored the thread.
Message ID: ***@***.***>
|
That worked perfectly. The error in metadata is gone. I test others movies and episodes that give me errors in the past and all have clean debug-info now. |
This issue can be closed after the new release. Thank you all for testing, it helped a lot. |
Congrats on new release! Started new discussion to restore it in homebrew: https://github.com/orgs/Homebrew/discussions/5459 |
@getzze I'm not very fluent in python and while trying to restore sublimal to homebrew I ran into issue with dependencies. I saw |
There is no consensus on a "dependency.lock" file, see the rejected PEP <https://peps.python.org/pep-0665/>
But you can use this tool for instance to generate such a list: <https://github.com/jazzband/pip-tools>
…On 25 June 2024 16:40:22 GMT+01:00, Wojtek ***@***.***> wrote:
@getzze I'm not very fluent in python and while trying to restore sublimal to homebrew I ran into issue with dependencies. I saw `pyproject.toml` file but it seems it only list major dependencies. Is there a way to get all (including transitive) dependencies that are required by subliminal? something akin "dependencies.lock" file with list of all resolved dependencies that required and it's versions? My google-foo doesn't return anything useful :(
--
Reply to this email directly or view it on GitHub:
#1083 (comment)
You are receiving this because you were mentioned.
Message ID: ***@***.***>
|
In the end I converted dependencies to poetry format just to easily get list of all transitive dependencies so I could proceed. A bit hacky but it got job done :-) |
I am creating this issue to keep track of users that can help testing the new release. Like this you can be notified when we land the release candidate.
The text was updated successfully, but these errors were encountered: