Code Monkey home page Code Monkey logo

Comments (8)

chocolateboy avatar chocolateboy commented on June 16, 2024

I can't reproduce this (yes, it's possible they're A/B testing or still rolling this out). In order to investigate, I'll need more details (this applies to any issue report), e.g.:

  • Operating System
  • Browser name/version
  • Userscript engine name/version
  • Userscript version
  • Relevant custom browser/engine/script settings
  • Other relevant addons/userscripts?
  • Logged in or not logged in?
  • Any errors in the web/browser console? βœ”

My guess is that it's a conflict with another addon (e.g.), so I'd rule that out first.

from userscripts.

re11ding avatar re11ding commented on June 16, 2024

Am not sure what you mean by platform, so I am going to assume Google?
Firefox Browser version 88.0
Tampermonkey 4.13.6136
Google DWIMages 2.3.2
I will need a description of what you mean by custom settings. I did not know the script was configurable.
I have disabled all other scripts and addons not just related to google but to everything except DWIMages and Tampermonkey. Still gives the error.
Happens both when logged in or not logged in.

So based on this, we know it's not a conflict with another script and/or add-on.
It's funny you mention that script specifically though. I am trying to aid the writer of that to fix the new A/B testing that they're applying. I made a very "hacky" means of fixing it before finding out about your addon that does the same thing and works regardless if on the A/B side. I would have actually tried fixing this script too, but yours is a lot more complex and I definitely can't wrap my head around it.

EDIT: Would you like a copy of the HTML, in case it is A/B testing that's affecting both my logged in and incognito mode?

from userscripts.

chocolateboy avatar chocolateboy commented on June 16, 2024

Am not sure what you mean by platform, so I am going to assume Google?

I've changed it to Operating System.

I will need a description of what you mean by custom settings.

Any relevant settings in Firefox, Tampermonkey, or the script.

Would you like a copy of the HTML?

Sure, you can email it to me at the address in my profile. Just incognito, though. Please don't send me anything which could expose your account details.

from userscripts.

re11ding avatar re11ding commented on June 16, 2024

Ah okay. Windows 10 Education Edition version 1709 (incapable of updating further due to functionality issues.)
All Tampermonkey settings are default (Injection mode was originally on instant due to that hacky script I made since it needed it to work, but that's since been reverted to the default before I reported results from earlier.) Script is completely unmodified. It would take far too long to list settings changes on Firefox, so to make it easier, I created a new profile where all settings are default and unchanged, installed the add-on, installed the script, and same issue exists. So it's not a Firefox settings conflict.

I would just list all the Firefox setting changes I made, but there's quite a bit about:configs that are different and it would be very difficult to list them all.

I will email you the incognito HTML in a bit then. I will have everything disabled in order for you to apply the script and see what's wrong.

EDIT: The file has been sent.

from userscripts.

chocolateboy avatar chocolateboy commented on June 16, 2024

Thanks, got it.

Try this.

from userscripts.

re11ding avatar re11ding commented on June 16, 2024

Hey nice work! Looks like it's working fine! If I may ask, what changed in the A/B testing to break the script so... selectively? I am still a programmer, just not familiar with javascript. Would love to expand my knowledge a bit, hence why I'm asking.

from userscripts.

chocolateboy avatar chocolateboy commented on June 16, 2024

The metadata appears to have switched from (simplified):

Mine

[
    { type: "image", index: 0, url: "https://example.com/1.jpg" },
    { type: "relatedSearches", links: [...] }, // no index, but consumes 1
    { type: "image", index: 2, url: "https://example.com/2.jpg" },
]

to:

Yours

[
    { type: "image", index: 0, url: "https://example.com/1.jpg" },
    { type: "advert", data: [...] }, // no index, and doesn't consume 1
    { type: "image", index: 1, url: "https://example.com/2.jpg" },
]

- which means an array can no longer be used to look up the metadata for an image based on its index. I switched to a Map keyed on the index property since the result indices may (now) be distinct from the array indices. This is cleaner and probably a bit faster than cloning/duplicating the structure of the original array, so it's an improvement even if they don't roll out this change.

As for why your metadata/results are different than mine: I'm not sure. It looks like there are some advertising-related nodes in your metadata that aren't in mine. I can't tell from the HTML and it might not be visible if you're using an ad-blocker. New advertising widgets, and possibly distinguishing the metadata that populates them from the metadata for real results, seems like the sort of thing an advertising company might experiment with, particularly when searching for something like "lawn mower".

from userscripts.

re11ding avatar re11ding commented on June 16, 2024

That's curious. I gave you the complete HTML with all addons and userscripts disabled, so I am unsure as to why this is the case. I wanted the raw to be as clean as possible. If there was some way to make it cleaner, I'm all ears on that one so we may better find the root of the cause.

Curiously, uBlock does block the elements on both local and online, so if you're using an adblocker yourself, this may be why you don't see it. I mention this specifically as I can see the ads myself on my copy of the local but only with uBlock disabled.

None the less, I appreciate you heavily for fixing it! Thanks for keeping this up to date!

from userscripts.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    πŸ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. πŸ“ŠπŸ“ˆπŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❀️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.