Change 519281 had a related patch set uploaded (by DLynch; owner: DLynch):
[mediawiki/extensions/MobileFrontend@master] Log editor switches to visualeditorfeatureuse

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

Change 519281 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Log editor switches to visualeditorfeatureuse

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

That's mobile done. Another patch for desktop will be forthcoming.

@DLynch how is this different from logging an abort_type of switchwith or switchwithout? Admittedly, that doesn't tell us which editor the user is switching to, but that's generally easy to infer.

Change 519498 had a related patch set uploaded (by Jforrester; owner: DLynch):
[mediawiki/extensions/MobileFrontend@wmf/1.34.0-wmf.11] Log editor switches to visualeditorfeatureuse

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

@Neil_P._Quinn_WMF That's something which has been subject to general code-atrophy over time. It only fires the abort for a full teardown editor-switch on desktop (so NWE isn't counted), and mobile has gradually wound up getting in-line with that so it's not logging the aborts because it's not navigating away. I figured putting in a consistent "this is feature-usage, and not an abstract thing tied to page lifecycle" would get us a less fragile metric going forward.

Change 519498 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@wmf/1.34.0-wmf.11] Log editor switches to visualeditorfeatureuse

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

Mentioned in SAL (#wikimedia-operations) [2019-06-27T20:31:19Z] <jforrester@deploy1001> Synchronized php-1.34.0-wmf.11/extensions/MobileFrontend/resources/dist: T221191: Log editor switches to visualeditorfeatureuse (duration: 00m 50s)

Change 519529 had a related patch set uploaded (by DLynch; owner: DLynch):
[mediawiki/extensions/VisualEditor@master] Log editor switches to visualeditorfeatureuse

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

Change 519530 had a related patch set uploaded (by DLynch; owner: DLynch):
[mediawiki/extensions/WikiEditor@master] Log editor switches to visualeditorfeatureuse

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

I found an extra little data-glitch in there -- when switching from WikiEditor to VisualEditor it wouldn't log it properly, because it wasn't updated to correctly see the in-page load. It would thus wait to log an abort until VE was done and triggered a page-reload, and it wouldn't log that abort correctly as being a switch away from WikiEditor (because it detected that purely based on whether the link to visual editing currently had focus).

So, @Neil_P._Quinn_WMF: all analytics data about switching to VE from WikiEditor from the last 3-ish years is suspect. 😁

In T221191#5290813, @DLynch wrote:

I found an extra little data-glitch in there -- when switching from WikiEditor to VisualEditor it wouldn't log it properly, because it wasn't updated to correctly see the in-page load. It would thus wait to log an abort until VE was done and triggered a page-reload, and it wouldn't log that abort correctly as being a switch away from WikiEditor (because it detected that purely based on whether the link to visual editing currently had focus).

So, @Neil_P._Quinn_WMF: all analytics data about switching to VE from WikiEditor from the last 3-ish years is suspect. 😁

Hahaha, wow, multiple good catches! This will definitely help us tracking switching behavior during the A/B test 👏

Out of curiosity, I checked the percentage of aborts in each editor that were switches, and it lines up with what you say (where desktop VE is the only editor logging any at all).

select
    event.platform as platform,
    event.editor_interface as editor,
    concat(round((
        sum(cast(event.abort_type like "switch%" as int)) * 100 / count(*)
    ), 1), "%") as switch_aborts
-- From the unsanitized data, so it only covers the last 90 days
from event.editattemptstep
where
    event.action = "abort" and
    -- Partition predicate to make Hive happy
    year > 0
group by event.platform, event.editor_interface

Change 519529 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Log editor switches to visualeditorfeatureuse

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

Change 519530 merged by jenkins-bot:
[mediawiki/extensions/WikiEditor@master] Log editor switches to visualeditorfeatureuse

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