ThePornDatabase / namer

Renames adult video files so that the plex/jellyfin plugins and stash script will match without user input. Runs server or command mode.
https://theporndb.net
184 stars 29 forks source link

add videohash support #116

Closed 4c0d3r closed 1 year ago

codecov-commenter commented 1 year ago

Codecov Report

Merging #116 (3b43979) into main (ad931d3) will decrease coverage by 0.21%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #116      +/-   ##
==========================================
- Coverage   87.20%   86.99%   -0.22%     
==========================================
  Files          34       34              
  Lines        3759     3759              
  Branches      543      543              
==========================================
- Hits         3278     3270       -8     
- Misses        271      277       +6     
- Partials      210      212       +2     
Impacted Files Coverage Δ
namer/command.py 81.60% <0.00%> (-3.84%) :arrow_down:
test/web/namer_web_pageobjects.py 86.06% <0.00%> (+0.81%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

DirtyRacer1337 commented 1 year ago

You still need ImageHash module to calculate distances, check tests

DirtyRacer1337 commented 1 year ago

also, it's builded already, https://github.com/peolic/videohashes/releases

4c0d3r commented 1 year ago

There's a comparison executable (haven't tried it yet), but I'm hoping to use that as well to determine distance/matches - copied it over.

DirtyRacer1337 commented 1 year ago

it's faster that run exe

4c0d3r commented 1 year ago

@DirtyRacer1337 even though it's built already, it wasn't built with GitHub actions, so the provenance of the executables can't be verified by the curious user. I'm moving the build here so that the source of the artifacts can be verified.

4c0d3r commented 1 year ago

Still a WIP but much improved installation and workflow, so merging. Also think the race condition in watchdog tests might be solved, else good logging will help identify the problem.