Very active discussion here: https://phabricator.wikimedia.org/T246901

Looks like an abandoned project. Feel free to reopen when it's reactivated.

Looks like an abandoned project. Feel free to reopen when it's reactivated.

Anything left to do here?

Change 835079 abandoned by Samwilson:

[mediawiki/extensions/PageImages@master] Add new {{#pageimage:}} parser function

Reason:

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

Mentioned in SAL (#wikimedia-operations) [2023-11-02T20:32:55Z] <mabualruz@deploy2002> Finished scap: Backport for [[gerrit:971244|Enable native math rendering mode on testwiki (T311620)]] (duration: 14m 06s)

Mentioned in SAL (#wikimedia-operations) [2023-11-02T20:20:11Z] <mabualruz@deploy2002> mabualruz and physikerwelt: Backport for [[gerrit:971244|Enable native math rendering mode on testwiki (T311620)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2023-11-02T20:18:49Z] <mabualruz@deploy2002> Started scap: Backport for [[gerrit:971244|Enable native math rendering mode on testwiki (T311620)]]

Change 971244 merged by jenkins-bot:

[operations/mediawiki-config@master] Enable native math rendering mode on testwiki

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

Change 971244 had a related patch set uploaded (by Physikerwelt; author: Physikerwelt):

[operations/mediawiki-config@master] Enable native math rendering mode on testwiki

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

Change 969510 merged by jenkins-bot:

[mediawiki/extensions/Math@master] Remove functional dead code about PNG images

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

Change 969508 merged by jenkins-bot:

[mediawiki/extensions/Math@master] Explain the significance of the img.tex css element

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

Change 969510 had a related patch set uploaded (by Physikerwelt; author: Physikerwelt):

[mediawiki/extensions/Math@master] Remove functional dead code about PNG images

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

Change 969508 had a related patch set uploaded (by Physikerwelt; author: Physikerwelt):

[mediawiki/extensions/Math@master] Explain the significance of the img.tex css element

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

Change 965923 abandoned by Physikerwelt:

[mediawiki/extensions/Math@master] Stop storing PNG images

Reason:

probably too big, split into several changes

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

From https://gerrit.wikimedia.org/r/q/Ib2c216f54e6817ee2c3be0355ba72bd4769ba6ea

Doesn’t MediaWiki keep track of patches it has already applied? Ouch. Then I guess we have to remove the old migration indeed, but I’d like to get a confirmation from James or someone else more familiar with the MW database handling.

Change 965923 had a related patch set uploaded (by Physikerwelt; author: Physikerwelt):

[mediawiki/extensions/Math@master] Stop storing PNG images

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

@Ladsgroup yes, that would be great.

https://github.com/wikimedia/restbase/pull/1311 has been open for a while. Should I go bother Virginia about this?

In T311620#8813089, @Ladsgroup wrote:

I guess it partially goes to production mathoid (through restbase?) and then partially to beta cluster which is problematic but oh well.

I guess it's partially goes to production mathoid (through restbase?) and then partially to beta cluster which is problematic but oh well.

In T311620#8810174, @Ladsgroup wrote:

The restbase is production but the actual mathoid is beta cluster: from my understanding https://github.com/wikimedia/mediawiki-extensions-Math/blob/master/extension.json#L122

		"MathMathMLUrl": {
			"value": "https://mathoid-beta.wmflabs.org"
		},

The restbase is production but the actual mathoid is beta cluster: from my understanding https://github.com/wikimedia/mediawiki-extensions-Math/blob/master/extension.json#L122

In T311620#8805725, @Ladsgroup wrote:

And the default is the mathoid in beta cluster that is completely unmaintained and unstable and goes down for weeks (services, mw is "just" days usually). People are already relying on an unstable API (infra-wise).

And the default is the mathoid in beta cluster that is completely unmaintained and unstable and goes down for weeks (services, mw is "just" days usually). People are already relying on an unstable API (infra-wise).

In T311620#8805581, @daniel wrote:

I do get user-facing APIs needing to be stable but this is internal API to mediawiki.

As someone who set up Math four years ago, how would you have known that?

I do get user-facing APIs needing to be stable but this is internal API to mediawiki.

I do get user-facing APIs needing to be stable but this is internal API to mediawiki. If someone wants png support for their own mw installation, they can setup up either their own mathoid service or cli mathoid. They already have a mw setup. To me, it's like having a service that provides memcached to 3rd party mw installations.

Pinging @VirginiaPoundstone. As the PM of API, I'd say it's up to her to decide what level of compatibility we need to retain.

In T311620#8805001, @daniel wrote:

In T311620#8804767, @Jdforrester-WMF wrote:

I don't agree. Can you please link to this task and mark it as a blocker?

It's on github: https://github.com/wikimedia/restbase/pull/1311

In T311620#8804804, @Ladsgroup wrote:

I agree with James. Mathoid API in wikimedia production is to serve Wikimedia wikis, not random 3rd party users. The default was set to point to wikimedia for ease of installation for development (which I don't even agree it was a good idea) not "here is some free service for you".

That's exactly what the documentation sais though: https://www.mediawiki.org/wiki/Extension:Math

In T311620#8804767, @Jdforrester-WMF wrote:

I don't agree. Can you please link to this task and mark it as a blocker?

I agree with James. Mathoid API in wikimedia production is to serve Wikimedia wikis, not random 3rd party users. The default was set to point to wikimedia for ease of installation for development (which I don't even agree it was a good idea) not "here is some free service for you".

In T311620#8804762, @daniel wrote:

Per discussion elsewhere: we need to retain some level of support for the png endpoints, since wmf is offering them as apis for use by the math extension on 3rd party wikis.

Per discussion elsewhere: we need to retain some level of support for the png endpoints, since wmf is offering them as apis for use by the math extension on 3rd party wikis.

Change 835295 abandoned by Nik Gkountas:

[mediawiki/extensions/PageImages@master] refactor: move utility methods from ParserFileProcessingHookHandlers

Reason:

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