Git Product home page Git Product logo

Comments (10)

judell avatar judell commented on May 27, 2024

I am investigating this. Another anomaly here, unsure if related or not, is that location order is unexpected. Manu's convention is to annotate the title so his summary appears first in the default location-sorted sidebar.

But here -- https://hyp.is/z7ElAlqIEeaB4898VeRkKg/www.theguardian.com/environment/2016/aug/02/environment-climate-change-records-broken-international-report -- it doesn't. Instead https://hyp.is/92NkkllsEea8JIP9CV7biQ/www.theguardian.com/environment/2016/aug/02/environment-climate-change-records-broken-international-report appears first, but anchors to the caption of the image, several elements below the title.

from client.

judell avatar judell commented on May 27, 2024

Observations

from client.

judell avatar judell commented on May 27, 2024

TextPositionSelectors don't agree with TextQuoteSelectors

Compare these two annotations:

0hTyrlltEeaVA9fVSqEUdw

                    {
                        "type": "TextPositionSelector",
                        "end": 4446,
                        "start": 4417
                    },
                    {
                        "exact": "more than 50% of its surface.",
                        "prefix": "grate, experienced melting over ",
                        "type": "TextQuoteSelector",
                        "suffix": "\nThe rapid changes in the climat"
                    }

"text" : "Like posted above ... "
"user" : leneae101

S8_CyFo_Eea52vcs3J9AVg

                    {
                        "type": "TextPositionSelector",
                        "end": 4431,
                        "start": 4402
                    },
                    {
                        "exact": "more than 50% of its surface.",
                        "prefix": "grate, experienced melting over ",
                        "type": "TextQuoteSelector",
                        "suffix": "\nThe rapid changes in the climat"
                    }

"text": "This is true. .."
"user" : "twilamoon"

The TextQuoteSelectors matches exactly, the TextPositionSelectors have the same length (29) but are offset by about half that length.

from client.

judell avatar judell commented on May 27, 2024

See also hypothesis/h#3278

from client.

robertknight avatar robertknight commented on May 27, 2024

Fix anchoring error on Guardian Climate Change article

from client.

sean-roberts avatar sean-roberts commented on May 27, 2024

This is very likely related https://via.hypothes.is/http://www.newyorker.com/science/maria-konnikova/being-a-better-online-reader

from client.

judell avatar judell commented on May 27, 2024

Here's a diff between an annotation that anchors in that maria-konnikova article and one that doesn't:

image

I know we've suspected that overlapping annotations may be a factor in some of the problems we've been seeing, this seems to point in that direction.

In that case, btw, the result is the same in via and the extension.

from client.

judell avatar judell commented on May 27, 2024

But here is a minimal version of the above that does not exhibit the problem:

http://jonudell.net/h/anchoring/anchoring-01.html

image

from client.

judell avatar judell commented on May 27, 2024

I'll admit I'm grasping at straws, but here is a possibly useful clue. I created a simpler page at http://jonudell.net/h/anchoring/konnikova_03.html and programmatically recreated the above annotations using only TextQuoteSelector:

            "selector": [
                {
                    "exact": "The text you read on a Kindle or computer simply doesn\u2019t have the same tangibility.",
                    "prefix": "rmer grounding in the material. ",
                    "type": "TextQuoteSelector",
                    "suffix": " In new research that she and he"
                }

and

         "selector": [
                {
                    "exact": "he text you read on a Kindle or computer simply doesn\u2019t have the same tangibility.",
                    "prefix": "mer grounding in the material. T",
                    "type": "TextQuoteSelector",
                    "suffix": " In new research that she and he"
                }
            ]

That page exhibits the problem: one anchors, another does not.

image

Then, taking a copy of that page at http://jonudell.net/h/anchoring/konnikova_04.html, I made those annotations interactively.

In that case, both anchor as expected:

image

Could this be a compound/cascading thing where, if TextPositionSelector is messed up (for unknown reasons), then the fallback to TextQuoteSelector also fails for overlap-related reasons? (hypothesis/h#3278)

from client.

robertknight avatar robertknight commented on May 27, 2024

One other issue I saw on the Guardian article is then activating, de-activating and then re-activating the extension with v0.47.0 of the client:

Uncaught Error: duplicate define: jquery
    at K (app.js:formatted:166)
    at F (app.js:formatted:173)
    at jquery:10269
    at deletedIds (jquery:26)
    at Object.require.jquery (jquery:38)
    at s (_prelude.js:1)
    at s (_prelude.js:1)
    at _prelude.js:1
    at Object.<anonymous> (annotator.js:2)
    at Object.32.jquery (annotator.js:1956)

from client.

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.