Page MenuHomePhabricator

Pass title to tone check API
Closed, ResolvedPublic

Description

when the model was trained, page title was used as an input to the model, in addition to the language code and the edit text. right now we're not passing in page title which could potentially result in a difference between the model's expected behavior vs. actual behavior. would it be possible for y'all to fetch and pass page title as an additional input from VE?

Details

Event Timeline

Change #1162981 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/VisualEditor@master] Pass page_title to tone check API

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

Test wiki created on Patch demo by ESanders (WMF) using patch(es) linked to this task:
https://patchdemo.wmcloud.org/wikis/7e2751a4d2/w/

Change #1162981 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] Pass page_title to tone check API

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

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

https://patchdemo.wmcloud.org/wikis/7e2751a4d2/w/