-
Notifications
You must be signed in to change notification settings - Fork 1
Switch from ImageMagick to VIPS #85
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 5.2.x
Are you sure you want to change the base?
Conversation
|
Apparently Extension:PdfHandler has a hard dependency to ImageMagick. |
root-fs/app/conf/LocalSettings.php
Outdated
| $GLOBALS['wgEnableUploads'] = true; | ||
| $GLOBALS['wgUploadPath'] = $GLOBALS['wgScriptPath'] . '/img_auth.php'; | ||
| $GLOBALS['wgUseImageMagick'] = true; | ||
| $GLOBALS['wgUseImageMagick'] = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a required settings by Extension:Thumbro? Telling from its usage of the BitmapHandlerTransformHook, which is called at https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/core/+/refs/heads/REL1_43/includes/media/TransformationalImageHandler.php#232 , scaler "hookaborted" will be used as an overwriting option regardless of the $wgUseImageMagick config value.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In fallback case https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/core/+/refs/heads/REL1_43/includes/media/BitmapHandler.php#596 one will use GD. According to https://github.com/libvips/libvips/wiki/Speed-and-memory-use GD is actually faster than ImageMagick? Not really sure here.
Apparently ImageMagick is very resource consuming. Using a tool like
vipsmay reduce the this.ERM43030
[5.x]