Page MenuHomePhabricator

RfC: Reducing image quality for mobile
Closed, ResolvedPublic

Description

Task to follow the progress of the RfC Reducing image quality for mobile.

Details

Reference
fl539
Related Changes in GitLab:
TitleReferenceAuthorSource BranchDest Branch
Removed dispatch method and supporting methods from JS clientrepos/data-engineering/metrics-platform!107sfaciT354822-remove-dispatch-methodmain
Add Codfw rack D2repos/data_persistence/swift-ring!15mvernoncodfw_d2main
Add codfw rack A4repos/data_persistence/swift-ring!11mvernoncodfw_a4main
Upgrade spark and icebergrepos/data-engineering/conda-analytics!56btullisupgrade_sparkmain
Add a staging airflow dag for webrequest_frontend refinementrepos/data-engineering/airflow-dags!892gmodenarevert-6cc86c0fmain
Use refinery job jar v0.2.43 containing fix for MediawwikiHistory duplicate checkerrepos/data-engineering/airflow-dags!744ebysansmw_dqmain
docker-hub-mirror: Use chart 1.7.0repos/releng/gitlab-cloud-runner!378dancymain-Ic3976a0a0e2478b35f4d2a95901dbadd1de54df3main
Chart 1.7.0: Add workaround for https://github.com/distribution/distribution/issues/2367repos/releng/docker-hub-mirror!34dancymain-I91e047b8a2f12671b77f6a6138acf15d0f195466main
Update function-schemata sub-module to HEAD (647a2cf)repos/abstract-wiki/wikifunctions/wikilambda-cli!36jforrestersync-function-schematamain
Update function-schemata sub-module to HEAD (647a2cf)repos/abstract-wiki/wikifunctions/function-evaluator!210jforrestersync-function-schematamain
Update function-schemata sub-module to HEAD (647a2cf)repos/abstract-wiki/wikifunctions/function-orchestrator!165apinesync-function-schematamain
Update analytics browser_metrics dagrepos/data-engineering/airflow-dags!666joalupdate_browser_report_dagmain
When canonicalize fails, have it return Z527 errorrepos/abstract-wiki/wikifunctions/function-schemata!112dmartinT354917main
When canonicalization fails, make the orchestrator return the errorrepos/abstract-wiki/wikifunctions/function-orchestrator!163dmartinT354917-2main
Draft: When result cannot be canonicalized, make orchestrator return Z527repos/abstract-wiki/wikifunctions/function-orchestrator!162dmartinT354917main
Update function-schemata sub-module to HEAD (5db8cea)repos/abstract-wiki/wikifunctions/function-orchestrator!158apinesync-function-schematamain
Update function-schemata sub-module to HEAD (a4a7edf)repos/abstract-wiki/wikifunctions/wikilambda-cli!35jforrestersync-function-schematamain
Update function-schemata sub-module to HEAD (a4a7edf)repos/abstract-wiki/wikifunctions/function-evaluator!202jforrestersync-function-schematamain
Update function-schemata sub-module to HEAD (a4a7edf)repos/abstract-wiki/wikifunctions/function-orchestrator!152jforrestersync-function-schematamain
builds-builder: bump to 0.0.95-20240403110641-057367b9repos/cloud/toolforge/toolforge-deploy!228ghostbump_builds-buildermain
Show related patches Customize query in GitLab

Event Timeline

flimport raised the priority of this task from to Medium.Sep 12 2014, 1:45 AM
flimport added a project: Architecture.
flimport set Reference to fl539.
daniel subscribed.

Discussed on 2014-04-16, (partially?) approved.

Minutes from https://www.mediawiki.org/wiki/Architecture_meetings/RFC_review_2014-04-16

  • ''LINK:'' https://www.mediawiki.org/wiki/Requests_for_comment/Reducing_image_quality_for_mobile (sumanah, 21:03:54)
  • I asked Yuri what he wanted: 1) an ok from ops to increase thumbnail storage by 2-3% and number of files by 15%, 2) from core/tim/etc to proceed with the proposed patch <yurik> assuming my proposed path is satisfactory to everyone's involved (sumanah, 21:04:15)
  • ''LINK:'' https://gerrit.wikimedia.org/r/#/c/119661/ Gerrit changeset, "Allow mobile to reduce image quality" (sumanah, 21:09:33)
  • comments were provided on the image quality gerrit patch (TimStarling, 21:42:33)
  • ''LINK:'' https://www.mediawiki.org/wiki/Talk:Requests_for_comment/Reducing_image_quality_for_mobile#File_insertion_syntax on wikitext addition (sumanah, 21:42:56)
  • image scaler backend relatively uncontroversial -- HTML/URL manipulation to access that API is more complex (TimStarling, 21:43:27)
  • gwicke predictably favours Node.JS service (TimStarling, 21:44:42)
  • <yurik> ok, all settled, will implement the first step (core patch), and start implementing JS magic (sumanah, 21:48:52)
  • required modifications: use string instead of integer "qlow-100px-image.jpg", make it JPG only (no png) (yurik, 21:50:31)
  • Tim skeptical about client-side JS rewrite: potential for CPU usage, flicker, image load aborts, browser incompatibilities, etc. (TimStarling, 21:54:34)