Page MenuHomePhabricator

Create Tone Check Patch demo for concept validation
Closed, ResolvedPublic

Description

This task involves the work of building a Patch demo prototype that the Design Research Team (specifically, @MRaishWMF) can use during the course of the interviews we will be conducting in T391162.

Requirements

Prioritized highest --> lowest

  1. Implement a probability score threshold so that a Peacock Check is shown only when the probability score is ≥0.8 or higher.
  2. Implement new copy, per T390248
  3. Suppress the dialog that prompts people to choose between the Real API and "amazing" Fake
  4. Fix issue where Peacock Check is shown if you simply add a space to a new paragraph
  5. Ensure the Peacock Check card is displayed during the Pre-Save moment if someone does not elect to Revise or Decline during the Mid-Edit moment.
    • At present, if someone skips the Peacock Check during the Mid-Edit moment, the Check is not resurfaced during Pre-Save.

Details

Event Timeline

Excited for this! Can we add a requirement about the demo implementing a probability score threshold (eg. only surfacing the peacock check when the probability score is 0.8 or higher)? This will allow us to demo something that is closer to what we expect the production experience to look like. We could also test different thresholds during these interviews if it makes sense to do so.

In T392871#10776491, @SSalgaonkar-WMF wrote:

Excited for this! Can we add a requirement about the demo implementing a probability score threshold (eg. only surfacing the peacock check when the probability score is 0.8 or higher)? This will allow us to demo something that is closer to what we expect the production experience to look like. We could also test different thresholds during these interviews if it makes sense to do so.

Oh, yes! Great spot, @SSalgaonkar-WMF. Let's do as you proposed: implement a probability score threshold so that a Peacock Check is shown only when the probability score is ≥0.8 or higher.

I've updated the task description to reflect this.

Change #1140499 had a related patch set uploaded (by Zoe; author: Zoe):

[mediawiki/extensions/VisualEditor@master] Make peacock check more demo-ready

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

  • Probability threshold implemented and set to 0.8
  • Implemented new copy
    • Note: Not for mid-edit spawned from pre-save – still looking in how to detect that case
  • Suppressed the dialog, switched to using the real API
  • Peacock check no longer shows on every change due to the 0.8 threshold

In progress:

  • Ensuring pre-save shows if mid-edit was ignored
  • Different messaging if the user came from pre-save

Fired up a patchdemo:

Screenshot 2025-05-01 at 14.44.35.png (582×1 px, 100 KB)

(removed the Research tag given that Zoe is working on it and this is not a request for Research to triage. Thanks.)

ppelberg moved this task from Doing to Ready for Sign Off on the Editing-team (Kanban Board) board.
ppelberg added a subscriber: zoe.

Next steps

  • @ppelberg to review patch demo
  • @zoe to file new ticket for requirements for "5."

Next steps

@zoe, https://patchdemo.wmcloud.org/wikis/d2d13f14a1/wiki/Main_Page is coming along nicely.

Beyond the "in progress" work [i] that will happen in a new ticket, I see one thing remaining to close this ticket out...

Could you pelaseto update the button copy within the Peacock Check card so that it reads Decline instead of Ignore? [ii]

Screenshot 2025-05-06 at 13.35.14.png (434×790 px, 37 KB)


i. In progress via T392871#10783331:

Ensuring pre-save shows if mid-edit was ignored
Different messaging if the user came from pre-save

ii. Note: the button copy might be fixed in the latest patch demo you seemed to have created (ef38198cb5). Although, I've not yet been able to get the Check to activate and subsequently, review the button copy.

I've updated the copy (without translation infrastructure) however I'm unable to create a patchdemo at present due to an error outside of my control – I'll try again in a little bit.

I've updated the copy (without translation infrastructure) however I'm unable to create a patchdemo at present due to an error outside of my control – I'll try again in a little bit.

Sounds great, @zoe. Let's track the change to revise Ignore to Decline in T390248.

And with work on requirement "5." now happening in T393570, let's consider this task Resolved.

Test wiki on Patch demo by Zoe-WMF using patch(es) linked to this task was deleted:

https://patchdemo.wmcloud.org/wikis/d2d13f14a1/w/

Test wiki on Patch demo by Zoe-WMF using patch(es) linked to this task was deleted:

https://patchdemo.wmcloud.org/wikis/ef38198cb5/w/

Change #1140499 abandoned by Zoe:

[mediawiki/extensions/VisualEditor@master] Make peacock check more demo-ready

Reason:

See I9d6efdfce20bed162ea20e6268bd14af6290c063

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

Aklapper renamed this task from Create Peacock Check Patch demo for concept validation to Create Tone Check Patch demo for concept validation.May 28 2025, 11:43 AM