Skip to content
Extraits de code Groupes Projets

Utiliser Intervention.io quand c'est possible pour que les filtre image puissent basculer de GD2 à Imagick selon la lib choisie dans l'interface privé

Ouvert cerdic a demandé de fusionner refactor_image_intervention vers master

Sur certains filtres on va parfois toujours utiliser GD2 pour des raisons de rapidité (par exemple |image_nb avec des arguments de pondération)

Rapports de requête de fusion

Loading
Loading

Activité

Filtrer l'activité
  • Approbations
  • Assignés et relecteurs
  • Commentaires (des bots)
  • Commentaires (des utilisateurs)
  • Branches et validations
  • Modifications
  • Labels
  • État de verrouillage
  • Mentions
  • État de la demande de fusion
  • Suivi
  • cpol0
    • Résolue par Maïeul

      relecture vite fait. Pas de spip5 sous la main, j'ai pas testé chez moi.

  • cerdic added 19 commits

    added 19 commits

    • 26394799...72614e93 - 9 earlier commits
    • 9f736ad1 - refactor: le filtre |image_fond_transparent utilise _image_map() pour une...
    • 8cbd0d8b - refactor: optimisation de _image_map() avec GD2, on evite d'instancier un...
    • e157762e - refactor: filtre image_alpha utilise intervention et _image_map
    • c3b3c9f5 - fix: antiblending plus robuste en gd
    • b0025f7f - refactor: le filtre image_flou implémente une version intervention pour quand...
    • 84a1bdba - refactor: le filtre |image_gamma est renomme |image_luminosite et implementé...
    • 78ee3f0c - feat: une fonction _image_colorize() applicable sur \Intervention\Image\Image,...
    • 62fc06d9 - feat: un filtre |image_colorize qui permet de modifier les composantes RGB...
    • 5bd4061f - refactor: fonction |image_sepia
    • fa817fd9 - refactor: le filtre |image_sepia a une implementation en GD et en Imagick,...

    Compare with previous version

  • cerdic added 3 commits

    added 3 commits

    • eb1dfd08 - refactor: le filtre |image_renforcement fonctionne avec Imagick comme avec GD2
    • ca20d54d - refactor: filtre image_masque, renommage de variables, preparation du masque...
    • e5966159 - refactor: filtre |image_masque utilise image intervention et fonctionne avec GD2 et Imagick

    Compare with previous version

  • cerdic added 4 commits

    added 4 commits

    • b3bd9d6f - refactor: simplification du decodage de la position du masque, utiliser...
    • 72aae8c2 - chores: variables inutiles
    • 6c03d17a - refactor: utiliser Image Intervention pour image_oriente_selon_exif() +...
    • 92686292 - refactor: utiliser image intervention pour |couleur_extraire

    Compare with previous version

  • cerdic marked this merge request as ready

    marked this merge request as ready

  • Auteur Maintainer

    Tous les filtres ont

    • soit une implémentation basée sur image intervention, qui fonctionne donc avec GD2 et Imagick
    • soit une implémentation alternative pour Imagick quand intervention ne permet pas de solution alternative. A noter que dans ce cas on utilise souvent image intervention pour manipuler l'image, mais selon que l'on utilise GD2 ou Imagick le traitement est différencié

    Seul le filtre |image_nb contient une variante d'implémentation (lorsque l'on spécifie les ponderations R/G/B) qui ne fonctionne qu'avec GD2, mais je suis pas certain que l'argument de rapidité vaille encore qu'on maintienne cette distinction et je suis assez pour basculer ça avec intervention aussi

    Le comportement des filtres avec Imagick vs GD2 est

    • en règle général identique ou très proche
    • dans le cas de |image_sepia on a cependant un rendu qui diffèren sensiblement selon la lib utilisée, mais qui semble acceptable dans les 2 cas
    • Résolue par b_b

      Seul le filtre |image_nb ... je suis assez pour basculer ça avec intervention aussi

      Alors ne pas hésiter :)

  • cerdic mentioned in merge request !4742

    mentioned in merge request !4742

  • cerdic added 3 commits

    added 3 commits

    • 2443492e - fix: apres une rotation avec imagick il faut reset la page sinon les crops a...
    • 6c1fedd7 - fix: output sur image_flou ne se faisait pas en cas d'image SVG
    • 51be7c29 - fix: catcher les exceptions à la lecture de l'image, et en cas d'echec...

    Compare with previous version

  • cerdic added 1 commit

    added 1 commit

    • b843a71c - fix: gerer les exceptions à la lecture de l'image via Imagick (format non...

    Compare with previous version

  • cerdic added 1 commit

    added 1 commit

    • f4d3373b - refactor: utiliser image intervention pour le calcul pixel a pixel de la...

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Veuillez vous inscrire ou vous connecter pour répondre
    Chargement en cours