Skip to content

Block AD in Large preview#1694

Merged
revonateB0T merged 18 commits intoFooIbar:mainfrom
revonateB0T:asdasdad
Sep 30, 2024
Merged

Block AD in Large preview#1694
revonateB0T merged 18 commits intoFooIbar:mainfrom
revonateB0T:asdasdad

Conversation

@revonateB0T
Copy link
Copy Markdown
Collaborator

No description provided.

@revonateB0T revonateB0T requested a review from FooIbar September 30, 2024 05:07
@revonateB0T revonateB0T marked this pull request as ready for review September 30, 2024 05:07
@revonateB0T
Copy link
Copy Markdown
Collaborator Author

1.Recompose has some problem
2.Try size original to detect
I will refine it later.

@revonateB0T revonateB0T marked this pull request as draft September 30, 2024 05:56
@revonateB0T revonateB0T marked this pull request as ready for review September 30, 2024 11:08
@FooIbar
Copy link
Copy Markdown
Owner

FooIbar commented Sep 30, 2024

The false negative rate is too high to justify shipping it.
Less than 1/4 images from the test gallery were detected.

@revonateB0T
Copy link
Copy Markdown
Collaborator Author

@FooIbar Then I have no idea, any further possibility to improve it?

@revonateB0T
Copy link
Copy Markdown
Collaborator Author

@revonateB0T If we could improve it later, we just land it as this MR doesn't touch rust code anyway.

@FooIbar
Copy link
Copy Markdown
Owner

FooIbar commented Sep 30, 2024

@FooIbar Then I have no idea, any further possibility to improve it?

I think detecting thumbnails is a dead end.
The URL of the large preview contains the original image's hash, maybe we can use that if we want to block ad thumbnails.

@revonateB0T revonateB0T merged commit 04ee1a6 into FooIbar:main Sep 30, 2024
FooIbar added a commit that referenced this pull request Oct 27, 2024
This reverts commit 04ee1a6

Reason for revert: Obsoleted by the upcoming changes
FooIbar added a commit that referenced this pull request Oct 27, 2024
This reverts commit 04ee1a6

Reason for revert: Obsoleted by the upcoming changes
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.

2 participants