Skip to content

Conversation

@Gykes
Copy link
Collaborator

@Gykes Gykes commented Nov 29, 2025

Currently, we have a filter for duplicate PHash. I revamped it a little bit to allow other duplicate checks. I kept PHash check but added in StashID and TItle. THe stashID is a current open feature request and the title is from me as I like to dedupe via titles.

Fixes: #3441

EDIT: I added in URl(s) as well per Smiths request it's just not in the img
UI:
Screenshot 2025-11-28 at 22 48 38
Screenshot 2025-11-28 at 22 48 49

@smith113-p
Copy link
Contributor

Would it be hard to also add URL to this?

@feederbox826
Copy link
Collaborator

Would it be hard to also add URL to this?

Dicussed this in VC, didn't see much use or practical application for URLs that wouldn't already be covered by previous filters

@Gykes
Copy link
Collaborator Author

Gykes commented Nov 29, 2025

@smith113-p I managed to get URls in there for you. This PR just got more complicated as I had to merge #6352 into this one.

@Gykes
Copy link
Collaborator Author

Gykes commented Dec 3, 2025

Deferring till after .30 release. Keeping open in the meantime.

@Gykes Gykes added this to the Version 0.31.0 milestone Dec 17, 2025
@Gykes
Copy link
Collaborator Author

Gykes commented Dec 17, 2025

I believe I will need to rework this some. With the URL changes being reverted from .30 I need to remove it from here as well.

@SikrytSquare
Copy link

I have been waiting for this for too long. Hope it gets merged soon. Huge thanks @Gykes

@discourse-stashapp
Copy link

This pull request has been mentioned on Stash. There might be relevant details there:

https://discourse.stashapp.cc/t/stash-v0-31-release-development-thread/4881/1

@discourse-stashapp
Copy link

This pull request has been mentioned on Stash. There might be relevant details there:

https://discourse.stashapp.cc/t/finding-duplicates-by-title/5085/2

@DogmaDragon DogmaDragon changed the title Feature: Duplicate Filter Feature: Scene Duplicate Filter Jan 16, 2026
@WithoutPants
Copy link
Collaborator

I think the graphql interface should be cleaned up a little. Here's my proposal:

  • rename existing PHashDuplicationCriterionInput to DuplicationCriterionInput
  • add optional boolean phash field. If present, scenes will be filtered based on if the phash is duplicated or not
  • add optional boolean url field. Same logic as above.
  • add optional boolean stash_id field. Same logic as above.
  • deprecate existing duplicated field, to be replaced with phash field - it should still maintain the legacy behaviour.
  • distance field should be documented to indicate that it is related to phash, but should also note that it is not implemented

This means that to filter on duplicate urls, you would have:

scene_filter: {
  duplicated: {
    url: true
  }
}

The UI code will need to be adapted to handle this style of criterion.

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

Successfully merging this pull request may close these issues.

[Feature] Add "Duplicated (Stash ID)" filter

6 participants