Maniphest T196380

Visual diff shows unchanged text containing a reference as removed/inserted (absent from source diff)
Closed, ResolvedPublic1 Estimated Story Points
Actions

Assigned To

Authored By

	Krinkle
	Jun 4 2018, 3:48 PM

Description

Comparing these two:

https://en.wikipedia.org/w/index.php?title=Ted_Nelson&type=revision&diff=842034627&oldid=832139178&diffmode=visual
https://en.wikipedia.org/w/index.php?title=Ted_Nelson&type=revision&diff=842034627&oldid=832139178&diffmode=source

The source diff shows 4 rough areas that were changed, with the last modified paragraph being:

- Nelson co-founded "itty bitty machine company", or "ibm", which was a small computer retail store operating from 1977 to 1980
+ In 1976, Nelson co-founded and briefly served as the advertising director of the "itty bitty machine company",

Screen Shot 2018-06-04 at 16.40.26.png (734×1 px, 238 KB)

The visual diff continues to show more differences, except they are not differences. The graphs were not in any way touched by the edits in between.

I do notice that the reference numbers have effectively changed due additions higher up in the document. However, those are usually shown in grey/blue with a note about them in the side bar. And in fact, in the left image, there is an example of that.

Details

	Subject	Repo	Branch	Lines +/-
	Update VE core submodule to master (3cb400800)	mediawiki/extensions/VisualEditor	master	+1 -1
	Performance improvements to compareElementsUnannotated	VisualEditor/VisualEditor	master	+91 -39

Customize query in gerrit

Related Objects

Mentioned In: T199572: Why does a single-paragraph, plain-old-text change produce a surprisingly messy diff?
Mentioned Here: rGVED3cb400800436: ce.Surface: afterPaste should always remove ve-attributes
rGVEDe333fe3275e3: Localisation updates from https://translatewiki.net.

Event Timeline

Krinkle created this task.Jun 4 2018, 3:48 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 4 2018, 3:48 PM

I get a diff timeout warning, which explains the full paragraph diffs, but not sure what is causing the timeout...

In profile, I notice there are some improvements we can make to compareElementsUnannotated, reducing it from >300ms to 20ms, but the diff still seems to being too much work at a higher level.

Change 437489 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] Performance improvements to compareElementsUnannotated

https://gerrit.wikimedia.org/r/437489

gerritbot added a project: Patch-For-Review.Jun 5 2018, 3:52 PM

With the above monkey patched in it dips below the timeout (just) and now looks more reasonable:

Deskana edited projects, added VisualEditor (Current work); removed VisualEditor.Jun 5 2018, 6:37 PM

Deskana assigned this task to Esanders.Jun 5 2018, 6:39 PM

Deskana triaged this task as Medium priority.

Deskana moved this task from Incoming to Code review on the VisualEditor (Current work) board.

Change 437489 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Performance improvements to compareElementsUnannotated

https://gerrit.wikimedia.org/r/437489

Change 438041 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (e333fe327)

https://gerrit.wikimedia.org/r/438041

Change 438041 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (3cb400800)

https://gerrit.wikimedia.org/r/438041

Jdforrester-WMF removed a project: Patch-For-Review.Jun 7 2018, 5:39 PM

Jdforrester-WMF moved this task from Code review to Product owner review on the VisualEditor (Current work) board.

Jdforrester-WMF added a project: MW-1.32-notes (WMF-deploy-2018-06-12 (1.32.0-wmf.8)).

Deskana closed this task as Resolved.Jun 11 2018, 3:05 PM

Jdforrester-WMF set the point value for this task to 1.Jun 13 2018, 6:24 PM

• Vvjjkkii renamed this task from Visual diff shows unchanged text containing a reference as removed/inserted (absent from source diff) to bobaaaaaaa.Jul 1 2018, 1:05 AM

• Vvjjkkii reopened this task as Open.

• Vvjjkkii removed Esanders as the assignee of this task.

• Vvjjkkii raised the priority of this task from Medium to High.

• Vvjjkkii added projects: CheckUser, Connected-Open-Heritage-Batch-uploads (RAÄ-KMB_1_2017-02), Tamil-Sites, Gamepress, Hashtags, Jade, KartoEditor, Language-2018-Apr-June, New-Editor-Experiences, Mail, TCB-Team (now WMDE-TechWish).

• Vvjjkkii updated the task description. (Show Details)

• Vvjjkkii removed the point value for this task.

• Vvjjkkii removed subscribers: gerritbot, Aklapper.

CommunityTechBot renamed this task from bobaaaaaaa to Visual diff shows unchanged text containing a reference as removed/inserted (absent from source diff) .Jul 2 2018, 7:21 AM

CommunityTechBot closed this task as Resolved.

CommunityTechBot assigned this task to Esanders.

CommunityTechBot lowered the priority of this task from High to Medium.

CommunityTechBot set the point value for this task to 1.

CommunityTechBot updated the task description. (Show Details)

CommunityTechBot removed projects: TCB-Team (now WMDE-TechWish), Mail, New-Editor-Experiences, Language-2018-Apr-June, KartoEditor, Jade, Hashtags, Gamepress, Tamil-Sites, Connected-Open-Heritage-Batch-uploads (RAÄ-KMB_1_2017-02), CheckUser.

CommunityTechBot added subscribers: gerritbot, Aklapper.

Schnark mentioned this in T199572: Why does a single-paragraph, plain-old-text change produce a surprisingly messy diff?.Jul 16 2018, 8:36 AM

	F18746171: Screen Shot 2018-06-04 at 16.40.26.png
	Jun 4 2018, 3:48 PM

	F18820226: image.png
	Jun 5 2018, 4:42 PM

Visual diff shows unchanged text containing a reference as removed/inserted (absent from source diff) Closed, ResolvedPublic1 Estimated Story PointsActions

Description

Details

Related Objects

Event Timeline

Visual diff shows unchanged text containing a reference as removed/inserted (absent from source diff)
Closed, ResolvedPublic1 Estimated Story Points
Actions