In T370610#10010516, @dcaro wrote:

Or to tell it that the size limit is unlimited (it just sets it to 0), @tstarling any preference on what big numbers to use to overwrite the limits?

In T370610#10010264, @dcaro wrote:

@tstarling I have a patch that implements the fix mentioned in the link above, do you have an image I can play with/test to make sure it works?

Bot updated. Template diff after editing "last updated" vs "created"

@aborrero tells me that the Debian maintainer is willing to remove the resource limits from policy.xml in the Debian package. That would be great to see, and will eventually help us to avoid production issues like T344233. It will also carry weight with Heroku if we decide to open a pull request there.

In T370230#9989106, @Samwilson wrote:

This might be true, but the main issue is that none of the fallbacks are applied, isn't it?

Fallback to zh is not ideal given the mutual intelligibility gap between Simplified and Traditional Chinese.

I can reproduce this with breakpoints set. onUpdateProject() is called first from the cdx-card @click handler, and then second from the cdx-checkbox @update:model-value handler.

In T369554#9975664, @GMikesell-WMF wrote:

@tstarling Please review issue below, thanks!
The title "Lorem ipsum" is below the "Last updated" title in the table, even though "Lorem ipsum" has a more recent date

In T223772#9975603, @Bugreporter wrote:

{{int:lang}} is a hack, and requires wikis creating some MediaWiki messages. See T4085: Add a {{USERLANGUAGE}} magic word

#time allows the user to specify an arbitrary language code, so I may as well extend that feature to the new function. The target keyword is not really necessary, since omitting the parameter would have the same effect. I can split the user keyword out to a separate commit.

Goals:

• Reasonably compact
• Encourage the use of the page language
• Give access to the user language, for the convenience of Commons where user and content language are conflated
• Give access to date, time and "both" formats

I installed AbuseFilter: https://wishlist-test.toolforge.org/wiki/Special:AbuseFilter

In T235397#9959778, @Tacsipacsi wrote:

What about creating a first version that doesn’t allow overriding any defaults (no changes to fuzzy units, no changes to priority languages, no toggling of page title translation, no toggling of translation-aware transclusion, no syntax version updates)? If I understand correctly, neither Community Tech nor @DannyS712’s bot needs the ability to override anything.

We need it for the community wishlist. The idea is to mark wishes for translation in a semi-automated way, with a gadget that runs as a translation admin (T365558). I don't think we need to override the defaults for the nofuzzy checkboxes.

Or there can be a nofuzzyunits parameter and a fuzzyunits parameter, the latter subtracting from the default set.

For API usability, I think an API request with action=markfortranslation & title=Foo should be equivalent to loading the form for that title and submitting it without changing any defaults. There's a couple of things in TranslatablePageMarker that make this inconvenient, like its need to overwrite the priority languages unconditionally. But it can be done.

The POST branch of PageTranslationSpecialPage::onActionMark() seems simple enough, but I assume a query API similar to showPage() will also be needed, and that looks a bit bigger. Unless it's enough to expect the client to guess the unit IDs for $noFuzzyUnits.

There's no permissions in the gadget implementation. You can't restrict who gets to change a status.

In T361067#9858226, @MusikAnimal wrote:

Random tech debt thing -- we have a few terms for whatever the subpage name for a wish/focus area is. Is slug a good name and if so maybe the db and backend things could be renamed? Or vice versa :)

@JScherer-WMF asked on Slack for image size statistics.

In T366824#9872926, @MarkAHershberger wrote:

I wish I understood how coroutines were disabled, but searching the Scribunto codebase found only a couple of references to the word coroutine in the tests. Fiddling ignorantly with those tests didn't seem to provide any meaningful change.

In T138933#9847347, @Sdkb wrote:

@tstarling I don't understand the gerritbot comment above. Would you be able to give an update on where this task is at?

TMH should set its own class attributes on the server side and avoid mangling unrelated content. Per my commit message comments on cf8a4f2133cb, TMH is too broken to use with Score. Requiring every other extension to set a fake TMH attribute to trick TMH into not breaking it doesn't seem like a good solution.

I just discovered T224245, which is a different approach to the same problem, already implemented.

In T201340#4508339, @Niharika wrote:

Do we want to do the row cleanup as part of this ticket? If not, let's get another ticket in.

Some usages of $wgBlockTargetMigrationStage remain in WMF-maintained extensions.

In T209892#9834400, @Clement_Goubert wrote:

@tstarling @dom_walden Is it possible to test an election on the mw-debug deployment of MW-on-K8s using XWD, or are we missing some configuration to try this in production?

In T352499#9813708, @Jdlrobson wrote:

FWIW We need some more background on why these messages were selected in the first case.

It seems to me that it should be possible to represent all powers of two up to the limit of the exponent, since the mantissa is just 1 for those numbers. So 2^53+1 is the lowest positive integer which needs to be rounded in order to fit in a double.

I suggest renaming bot.js to update-indexes.js.

In T361067#9790933, @MusikAnimal wrote:

Proposed implementation plan:

• Use sqlite3 for data persistence, using the filesystem to avoid the need for an SSH tunnel to a Cloud Services database (or local MySQL-like dependency)

This was fixed.

@aaron disabled it on 2013-03-14 and re-enabled it for Commons only on 2013-03-29. But there was no UI for it on wikis other than Commons, and that remains the case.

In T354949#9790843, @Ligliotoi wrote:

Hallo,

https://panoviewer.toolforge.org/ does not work. Error 403 "Forbidden". Can somebody fixed it maybe?

Confirmed fix on ar.wikipedia.org

In T346293#9785650, @matmarex wrote:

Looking here: https://gerrit.wikimedia.org/g/mediawiki/core/+/bc11d8072a47fdc2f9d3a60f78c09716948b626e/includes/block/DatabaseBlockStore.php#336 tables does not list block_target, but it refers to bt_* fields from that table.

The query definition a few lines below: https://gerrit.wikimedia.org/g/mediawiki/core/+/bc11d8072a47fdc2f9d3a60f78c09716948b626e/includes/block/DatabaseBlockStore.php#367 does list block_target when using fields from that table.

I'm not sure if that's a mistake, but it looks suspicious.

Temporary repo with unreviewed code: https://github.com/tstarling/pano-projector

In T363891#9762699, @tstarling wrote:

In T363891#9760677, @TheDJ wrote:

Panellum falls back to a low resolution full size image if the results are not yet ready. The toolforge tool also renders on the fly i believe and it will 'fill in' results as they arrive based on a JS poll.

There's no Pannellum support for that, it's only in the panoviewer tool.

I did a rough prototype of this in C++, and I got it down to 5.9 seconds for single-threaded extraction of a single cube face. So I think this is the way to go. I'll write a custom cube face extractor, and Thumbor will run it on demand, no need for a job.

The frequency of these errors before March 6 g1008752 reflected the frequency of Redis connection timeout errors. About 1 in every 160 errors raised by excimerFlushToArclamp() would fail with "HookRunner not found". So following Timo's patch, we can expect this to be very rare indeed. In logstash since that date, there has only been one such error. So I think it is a low priority.

DENIVIP's panorama converter shows how simple it can be. It does all 6 cube faces in 67 seconds, single threaded. They messed up the interpolation somehow, the CImg library they used is weird and scary, and the dependencies are wrong (it doesn't really need xorg-dev), but it's certainly inspiring. It's basically doing the thing we need in 229 lines of code.

By specifying bilinear interpolation I was able to reduce the time taken by PTmender to extract a cube face from 42 to 33 seconds. By default it uses a cubic polynomial on a 4x4 region of the input image, but I think that's excessive given we're only upscaling by a factor of between 1 and 2.

I made subtasks for what I see as the three main components of spherical panorama support:

In T363891#9760677, @TheDJ wrote:

Where would we store the results ? Make a new file storage subspace like score-render ?

In T345334#9752173, @Ladsgroup wrote:

That'd work on overall hits, as you said "sort images by popularity". That's not the case here. Front caches absorb all of the hits and collapse the power law distribution.

In T363885#9760636, @TheDJ wrote:

In my experiments so far, i've noticed that we also have a LOT of images that do NOT have GPano metadata.

In T363885#9760140, @AntiCompositeNumber wrote:

It only supports TIFF output, so it's necessary to pass the resulting image through ImageMagick to convert it to a JPEG.

What input formats are supported?

Following the merge of https://gerrit.wikimedia.org/r/1003091 it would be good if @dom_walden could do a test election using the new OpenSSL encryption method. See my local test log comment on PS5 for information about generating keys and creating an election.

We can't, due to T196132: wikidiff2 binaries are not distributable due to PHP / GPL license incompatibilities

In T345334#9654752, @Ladsgroup wrote:

If we do extrapolation after 10,000th hit. The Theil-Sen extrapolation becomes more useful:
So the simplest is to see is boundary problem and say it's 130 hits at zero and zero at 200M and draw a line.

Should be fixed in Shellbox 4.0.1 which is required by MediaWiki 1.42.