theotherp / nzbhydra2

Usenet meta search
Other
1.27k stars 76 forks source link

Grouping enhancement #759

Open DarrenPIngram opened 2 years ago

DarrenPIngram commented 2 years ago

Would you consider adding an enhancement/option to the grouping of search results, e.g. group by identified resolution, even if just at the 720, 1080 level, ignoring the different groups and names that exist.

Situation: At times I can search for a title, it may master group as

Almost.Never.S01E09.NORWEGiAN.720p.WEB.h264-NORKiDS

But beneath there are many different versions and if you wanted to "select all" you get more work, or a Norwegian version!

Almost.Never.S01E09.INTERNAL.720p.HEVC.x265-MeGusta nzb.su  TV HD   326.45 MB   5   1094d     
Almost.Never.S01E09.The.Chicken.iP.WEB-DL.AAC2.0.x264-BTN   nzbplanet   TV SD   648.02 MB   15  1122d      
Almost.Never.S01E09.The.Chicken.720p.iP.WEB-DL.AAC2.0.x264-BTN  nzbplanet   TV HD   1.12 GB 22  1122d      
Almost.Never.S01E09.NORWEGiAN.720p.WEB.h264-NORKiDS Drunken Slug    TV HD   552.42 MB   2   202d      
Almost.Never.S01E09.INTERNAL.720p.WEB.h264-WEBTUBE  Drunken Slug    TV HD   1.2 GB  1   1094d     
Almost.Never.S01E09.The.Chicken.iP.WEB-DL.AAC2.0.x264-BTN   Drunken Slug    TV HD   648.25 MB   1   1117d     
Almost.Never.S01E09.The.Chicken.720p.iP.WEB-DL.AAC2.0.x264-BTN

I don't pretend to have the logic model, but matching even 720p and 1080p (not here) and if they exist "expand" OR look for common nationalities NORWEGIAN, SWEDISH, IRISH or what have you and if they exist then "auto expand" may be an option.

theotherp commented 2 years ago

If enabled in the display options TV releases are already grouped by season and episode.

Parsing other releases like movies a lot more work and it's really complicated to get right.

DarrenPIngram commented 2 years ago

Thank you for the prompt reply. In the example given it was a result of the (TV) category search and then the items stated were cut and pasted from the application.

If some form of logging helps please say what exactly is necessary and I will try and assist.

D

On Tue, 15 Feb 2022 at 15:11, theotherp @.***> wrote:

If enabled in the display options TV releases are already grouped by season and episode.

Parsing other releases like movies a lot more work and it's really complicated to get right.

— Reply to this email directly, view it on GitHub https://github.com/theotherp/nzbhydra2/issues/759#issuecomment-1040254736, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV5MAEVW7VN6A3ZQM3TMEDU3JGHBANCNFSM5OOGPWKA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

albino1 commented 2 years ago

Can you not just use the Quick Filter selectors at the top of the results? There's already a 720p filter for example, and you can just create your own custom filter called Norwegian if you want in Config -> Searching -> Quick Filters.

DarrenPIngram commented 2 years ago

Hi. Sometimes that is appropriate. Not always.

The point is that the grouping could be "finer" as it is matching part of the string. It is just an issue when you search a large series and get a few dozen relevant hits, all grouped (which is good, to move it from being a few hundred hits) but then the language-specific versions always seem to take over, so you have to click the twisty and effectively reverse the work in any case.

Anyway, it is a clear first-world problem.

Thanks.

On Tue, 15 Feb 2022 at 23:10, albino1 @.***> wrote:

Can you not just use the Quick Filter selectors at the top of the results? There's already a 720p filter for example, and you can just create your own custom filter called Norwegian if you want in Config -> Searching -> Quick Filters.

— Reply to this email directly, view it on GitHub https://github.com/theotherp/nzbhydra2/issues/759#issuecomment-1040795808, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV5MAH6OD4CG4UUA56VX6DU3K6LVANCNFSM5OOGPWKA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

theotherp commented 2 years ago

As I said this isn't as easy as it may sound. There are whole (python) libraries to parse titles and guess stuff like title, season, episode. I don't think I can / will do this but I'll keep the issue open - who knows.

DarrenPIngram commented 2 years ago

OK thanks. I don't pretend to know the programming side of things, but it was particularly noticeable that the country/language names would sit at the top of the group.

I guess when you have a great experience with the application that you kindly work on, you tend to notice even small issues as the other stuff "just works" :=)

On Sun, 20 Feb 2022 at 13:00, theotherp @.***> wrote:

As I said this isn't as easy as it may sound. There are whole (python) libraries to parse titles and guess stuff like title, season, episode. I don't think I can / will do this but I'll keep the issue open - who knows.

— Reply to this email directly, view it on GitHub https://github.com/theotherp/nzbhydra2/issues/759#issuecomment-1046211852, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV5MAFUSFOQGRA4FABCLD3U4DCV3ANCNFSM5OOGPWKA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

theotherp commented 2 years ago

Check this file to see the formats supported by the mentioned library https://github.com/guessit-io/guessit/blob/develop/guessit/test/episodes.yml

On Sun, 20 Feb 2022, 12:03 DarrenPIngram, @.***> wrote:

OK thanks. I don't pretend to know the programming side of things, but it was particularly noticeable that the country/language names would sit at the top of the group.

I guess when you have a great experience with the application that you kindly work on, you tend to notice even small issues as the other stuff "just works" :=)

On Sun, 20 Feb 2022 at 13:00, theotherp @.***> wrote:

As I said this isn't as easy as it may sound. There are whole (python) libraries to parse titles and guess stuff like title, season, episode. I don't think I can / will do this but I'll keep the issue open - who knows.

— Reply to this email directly, view it on GitHub < https://github.com/theotherp/nzbhydra2/issues/759#issuecomment-1046211852 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/ABV5MAFUSFOQGRA4FABCLD3U4DCV3ANCNFSM5OOGPWKA

. Triage notifications on the go with GitHub Mobile for iOS < https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675

or Android < https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .

You are receiving this because you authored the thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/theotherp/nzbhydra2/issues/759#issuecomment-1046212253, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADNUA6M3PQRRTAALMIMWOP3U4DC67ANCNFSM5OOGPWKA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you commented.Message ID: @.***>

DarrenPIngram commented 2 years ago

Wasn't exactly clear about interpreting this to be honest.

I've had another thought.

There is already the ability to add filter words to an already made search. Could there be a user preference setting where the user enters a list of words, whether comma separated or through an input box one at a time (whatever's easiest) and then either a preference option OR a checkbox next to the filter to "filter pre-stored list"? The list remains set (or added/removed) within the application preferences and the user can choose to filter or not based on demand.

Then it would allow the user to put in words like NORWEGIAN, SWEDISH or whatever and then filter out (easier if you have a list of say 50 countries or terms.

Would that work/be easy (or easier) to consider implementing?

On Sun, 20 Feb 2022 at 13:53, theotherp @.***> wrote:

Check this file to see the formats supported by the mentioned library

https://github.com/guessit-io/guessit/blob/develop/guessit/test/episodes.yml

On Sun, 20 Feb 2022, 12:03 DarrenPIngram, @.***> wrote:

OK thanks. I don't pretend to know the programming side of things, but it was particularly noticeable that the country/language names would sit at the top of the group.

I guess when you have a great experience with the application that you kindly work on, you tend to notice even small issues as the other stuff "just works" :=)

On Sun, 20 Feb 2022 at 13:00, theotherp @.***> wrote:

As I said this isn't as easy as it may sound. There are whole (python) libraries to parse titles and guess stuff like title, season, episode. I don't think I can / will do this but I'll keep the issue open - who knows.

— Reply to this email directly, view it on GitHub <

https://github.com/theotherp/nzbhydra2/issues/759#issuecomment-1046211852

, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ABV5MAFUSFOQGRA4FABCLD3U4DCV3ANCNFSM5OOGPWKA

. Triage notifications on the go with GitHub Mobile for iOS <

https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675

or Android <

https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub

.

You are receiving this because you authored the thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub < https://github.com/theotherp/nzbhydra2/issues/759#issuecomment-1046212253 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/ADNUA6M3PQRRTAALMIMWOP3U4DC67ANCNFSM5OOGPWKA

. Triage notifications on the go with GitHub Mobile for iOS < https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675

or Android < https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .

You are receiving this because you commented.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/theotherp/nzbhydra2/issues/759#issuecomment-1046220339, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV5MAGXNPATH3THZYHLTLLU4DI2RANCNFSM5OOGPWKA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>