Page MenuHomePhabricator

Report language version in the metadata map & dialog
Closed, ResolvedPublic

Description

Description

When a function call is performed, using a supported-language implementation (Python, JavaScript, etc.), record the language version (e.g., Python 3.10.8) in the metadata map (Z22K2), and show it whenever the corresponding metadata dialog is displayed.

Note: It'll make sense to do this after settling the relevant operational details of the deployment of evaluators and executors.

Desired behavior/Acceptance criteria (returned value, expected error, performance expectations, etc.)

  • Language version should be recorded and displayed, as described above, whenever a supported-language implementation is executed.

Completion checklist

Event Timeline

cmassaro changed the task status from Open to In Progress.Oct 4 2023, 12:07 PM
cmassaro claimed this task.
cmassaro moved this task from Backlog to In Progress on the Abstract Wikipedia team board.

Change 964086 had a related patch set uploaded (by Jforrester; author: Cory Massaro):

[mediawiki/extensions/WikiLambda@master] Add programmingLanguageVersion metadata key to i18n.

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

Change 964086 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Add programmingLanguageVersion metadata key to i18n.

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

Works in wikifunctions beta, e.g.

Screen Shot 2023-10-11 at 11.38.45 AM.png (1×1 px, 216 KB)
https://wikifunctions.beta.wmflabs.org/view/en/Z10054

Translations are in place too (but not displayed in beta): MediaWiki:wikilambda-functioncall-metadata-execution-programming-language-version.

wikifunctions wmf.30 does not display the language version, e.g. https://www.wikifunctions.org/view/en/Z10065

Screen Shot 2023-10-11 at 11.56.32 AM.png (1×1 px, 290 KB)

Change 966896 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/extensions/WikiLambda@master] Follow-up c9a1cded: Add new i18n key to manifest too

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

Change 966896 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Follow-up c9a1cded: Add new i18n key to manifest too

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

Checked in wikifunctions beta (https://wikifunctions.beta.wmflabs.org/view/en/Z10054)

  • the language version is displayed with a correct label
before the fixafter the fix
Screen Shot 2023-10-11 at 11.38.45 AM.png (1×1 px, 216 KB)
Screen Shot 2023-10-19 at 3.36.57 PM.png (1×1 px, 203 KB)

In wikifunctions wmf.1 (https://www.wikifunctions.org/view/en/Z10065) the label is displayed, but not the Programming language version message text:

Screen Shot 2023-10-19 at 3.12.35 PM.png (1×1 px, 225 KB)

Jdforrester-WMF subscribed.

In wikifunctions wmf.1 (https://www.wikifunctions.org/view/en/Z10065) the label is displayed, but not the Programming language version message text:

Yup, the code was added in https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikiLambda/+/964086 but we forgot to add it to the manifest, fixed in https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikiLambda/+/966896 (which went out in 1.42.0-wmf.2).