Also ensure that all validators for Unit are only looking for instances of Z21 (I believe we currently allow both in many places).
Description
Details
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | DMartin-WMF | T285433 Replace Z23/Nothings with Z24/voids (instances of Z21/Unit) in Orchestrator and Validator | |||
Resolved | cmassaro | T287968 Create a makeUnit() Method in `function-schemata` | |||
Resolved | DVrandecic | T287195 Function model: Z21 should be updated to be the null value of Unit | |||
Resolved | DVrandecic | T276721 ZObject model: Don’t make Z23/nothing’s Z1K1/type itself |
Event Timeline
Currently we have the following discordance between the meaning of Z21, Z23 and Z24:
In documentation page:
Z21: Unit (Type).
Z23: Nothing (Type).
Z24: void (Unit).
In function-schemata data definitions:
Z21: void (Unit).
Z23: Nothing (Type).
Z24: Unit (Type).
In function-schemata normal and canonical yamls:
Z21: Unit (Type).
Z23: Nothing (Type).
Z24: void (Unit).
In WikiLambda:
Z23: Nothing (Type).
In function-schemata utils.js:
Z23: Unit (Type).
In function-schemata schema.js:
Z23: Unit (Type).
Change 776354 had a related patch set uploaded (by David Martin; author: David Martin):
[mediawiki/services/function-schemata@master] Replace Z23s with Z21s (instances of Z24) in function-schemata
Change 776397 had a related patch set uploaded (by David Martin; author: David Martin):
[mediawiki/services/function-schemata@master] Rename validator function & implementation for Z21
Change 776441 had a related patch set uploaded (by David Martin; author: David Martin):
[mediawiki/services/function-orchestrator@master] Replace Z23s with Z21s (instances of Z24) in function-orchestrator
Change 776461 had a related patch set uploaded (by David Martin; author: David Martin):
[mediawiki/services/function-evaluator@master] Replace Z23s with Z21s (instances of Z24) in function-evaluator
Change 776354 merged by jenkins-bot:
[mediawiki/services/function-schemata@master] [BREAKING CHANGE] Replace Z23s with Z24s in function-schemata
Change 776397 merged by jenkins-bot:
[mediawiki/services/function-schemata@master] definitions: Rename validator function & implementation for Z21
Change 776441 merged by jenkins-bot:
[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (3db774b) and replace Z23s with Z21s
Change 776990 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/extensions/WikiLambda@master] Update function-schemata sub-module to HEAD (3db774b)
Change 776461 merged by jenkins-bot:
[mediawiki/services/function-evaluator@master] Update function-schemata sub-module to HEAD (3db774b) and replace Z23s with Z21s
Change 778562 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (7aabc4a)
Change 778565 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/services/function-evaluator@master] Update function-schemata sub-module to HEAD (4fdbbf4)
Change 778566 had a related patch set uploaded (by Cory Massaro; author: Cory Massaro):
[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (4fdbbf4)
Change 778562 merged by jenkins-bot:
[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (4fdbbf4)
Change 776990 merged by jenkins-bot:
[mediawiki/extensions/WikiLambda@master] Update function-schemata sub-module to HEAD (4fdbbf4)
Change 778565 merged by jenkins-bot:
[mediawiki/services/function-evaluator@master] Update function-schemata sub-module to HEAD (4fdbbf4)
Change 778566 abandoned by Jforrester:
[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (4fdbbf4)
Reason:
Sorry, this is now a no-op!
Change 779748 had a related patch set uploaded (by David Martin; author: David Martin):
[mediawiki/services/function-orchestrator@master] Replace Z23s with Z24s in function-orchestrator
Change 779748 merged by jenkins-bot:
[mediawiki/services/function-orchestrator@master] Replace Z23s with Z24s in function-orchestrator
Change 780904 had a related patch set uploaded (by David Martin; author: David Martin):
[mediawiki/services/function-evaluator@master] Replace Z23s with Z24s in function evaluator
Change 781605 had a related patch set uploaded (by David Martin; author: David Martin):
[mediawiki/services/function-orchestrator@master] Use makeResultEnvelopeWithVoid in orchestrator
Change 781645 had a related patch set uploaded (by David Martin; author: David Martin):
[mediawiki/services/function-schemata@master] Finish replacing Z23s with Z24s in function-schemata
Change 781605 merged by jenkins-bot:
[mediawiki/services/function-orchestrator@master] Use makeResultEnvelopeWithVoid in orchestrator
Change 780904 merged by jenkins-bot:
[mediawiki/services/function-evaluator@master] Replace Z23s with Z24s in function evaluator
For future reference, https://gerrit.wikimedia.org/r/781605 added argument withVoid to function-schemata methods normalize and canonicalize. Setting withVoid = true causes these methods to use Z24 / void in their result envelopes. The default value of withVoid is false. When it's time for cleanup, we can change that default to be true.
Change 784732 had a related patch set uploaded (by David Martin; author: David Martin):
[mediawiki/services/function-orchestrator@master] Use -withVoid=true in calls to normalize and canonicalize
Change 781645 merged by jenkins-bot:
[mediawiki/services/function-schemata@master] Finish replacing Z23s with Z24s in function-schemata
Change 785853 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (49ff68e)
Change 785854 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/services/function-evaluator@master] Update function-schemata sub-module to HEAD (49ff68e)
Change 785856 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/extensions/WikiLambda@master] Update function-schemata sub-module to HEAD (49ff68e)
Change 785853 merged by jenkins-bot:
[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (49ff68e)
Change 784732 merged by jenkins-bot:
[mediawiki/services/function-orchestrator@master] Use withVoid=true in calls to normalize and canonicalize
Change 785854 merged by jenkins-bot:
[mediawiki/services/function-evaluator@master] Update function-schemata sub-module to HEAD (49ff68e)
Change 785856 merged by jenkins-bot:
[mediawiki/extensions/WikiLambda@master] Update function-schemata sub-module to HEAD (49ff68e)
Change 799366 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/services/function-schemata@master] Drop validatesAsUnit(), not used anywhere any more
Change 799366 merged by jenkins-bot:
[mediawiki/services/function-schemata@master] Drop validatesAsUnit(), not used anywhere any more
Change 800790 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/services/function-evaluator@master] Update function-schemata sub-module to HEAD (2575d10)
Change 800791 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (2575d10)
Change 800792 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/extensions/WikiLambda@master] Update function-schemata sub-module to HEAD (2575d10)
Change 800791 merged by jenkins-bot:
[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (2575d10)
Change 800790 merged by jenkins-bot:
[mediawiki/services/function-evaluator@master] Update function-schemata sub-module to HEAD (2575d10)
Change 799369 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/services/function-schemata@master] Drop makeResultEnvelope and makeUnit
Change 799369 merged by jenkins-bot:
[mediawiki/services/function-schemata@master] Drop makeResultEnvelope and makeUnit
Change 800792 merged by jenkins-bot:
[mediawiki/extensions/WikiLambda@master] Update function-schemata sub-module to HEAD (2575d10)
Change 802800 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/extensions/WikiLambda@master] Update function-schemata sub-module to HEAD (31f931f)
Change 802801 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (31f931f)
Change 802802 had a related patch set uploaded (by Jforrester; author: Jforrester):
[mediawiki/services/function-evaluator@master] Update function-schemata sub-module to HEAD (31f931f)
Change 802800 merged by jenkins-bot:
[mediawiki/extensions/WikiLambda@master] Update function-schemata sub-module to HEAD (5c8c897)
Change 802801 merged by jenkins-bot:
[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (5c8c897)
Change 802802 merged by jenkins-bot:
[mediawiki/services/function-evaluator@master] Update function-schemata sub-module to HEAD (5c8c897)
apine updated https://gitlab.wikimedia.org/repos/abstract-wiki/wikifunctions/function-orchestrator/-/merge_requests/29
Remove remaining references to Z23 in the orchestrator.
jforrester merged https://gitlab.wikimedia.org/repos/abstract-wiki/wikifunctions/function-orchestrator/-/merge_requests/29
Remove remaining references to Z23 in the orchestrator.