[Suggestion] Un Zoom plus Zen

**pseudocode** · 07/01/2010, 00h43

Tout d'abord, bravo pour la v2.0 : c'est beau !

(d'ailleurs, je compte donner un coup de menage dans le code de "mes" plugins pour la v2.0

)

Bon, concernant ma suggestion : peut-on faire quelque chose pour la fonction zoom:

- un zoom de meilleur qualité (genre celui du plugin "redimensionnement texte"), car lors d'une réduction on perd des infos dans l'image, par exemple des morceaux de lignes fines
- un zoom régulier avec la molette de la souris, pour que l'on se retrouve a 100% si on fait un zoom suivi d'un dézoom
- un zoom par defaut a 100% si l'image tient dans la fenetre

Voila, c'était quelques remarques en vrac.

**Jack Sparrow** · 07/01/2010, 08h03

Envoyé par pseudocode

- un zoom régulier avec la molette de la souris, pour que l'on se retrouve a 100% si on fait un zoom suivi d'un dézoom

Je crois que je n'ai pas compris ce point là

Est-ce que tu veux dire que si tu es à 100%, dézoomer puis zoomer (ou zoomer et dézoomer) avec la molette implique le retour à 100% ?
EDIT : Ah oui, ça déconne, ce n'est pas une feature

Pour le zoom de meilleur qualité, il faut que je vois comment optimiser ça.

Normalement, au niveau des anciens plugins, il n'y a pas grand chose qui change. A part avec la preview, les anciens devraient toujours fonctionner.

Je n'ai pas tout documenter car certaines parties n'ont pas des spécifications stables, mais il est possible d'ajouter son propre système de plugin, j'avais commencé à faire un plugin pour integrer les plugins ImageJ (sans avoir besoin de recompiler quoique ce soit).

**pseudocode** · 07/01/2010, 09h15

Envoyé par Jack Sparrow

Je crois que je n'ai pas compris ce point là

Est-ce que tu veux dire que si tu es à 100%, dézoomer puis zoomer (ou zoomer et dézoomer) avec la molette implique le retour à 100% ?
EDIT : Ah oui, ça déconne, ce n'est pas une feature

Pour le zoom de meilleur qualité, il faut que je vois comment optimiser ça.

Remarque, les 2 choses sont liées. C'est parce que le zoom fait "sauter" des pixels (exemple avec cette image) que j'aimerai bien pouvoir retourner a 100% avec la molette.

Normalement, au niveau des anciens plugins, il n'y a pas grand chose qui change. A part avec la preview, les anciens devraient toujours fonctionner.

C'est surtout du nettoyage pour optimisation (utiliser ton "flou gaussien rapide" dans les filtres) et du cosmetique (nom/type des parametres). Peut-être aussi virer des vieux filtres inutiles (UnNoise ?) et fusionner des filtres (genre les diffusions).

**pseudocode** · 07/01/2010, 13h35

Envoyé par Jack Sparrow

Normalement, au niveau des anciens plugins, il n'y a pas grand chose qui change. A part avec la preview, les anciens devraient toujours fonctionner.

Ca fonctionne pas toujours. Mise a part que GenericPluginFilter est Deprecated, il semble que le système de notification (notifyProcessFilter) n'existe plus.

**Jack Sparrow** · 07/01/2010, 13h44

Envoyé par pseudocode

Ca fonctionne pas toujours. Mise a part que GenericPluginFilter est Deprecated, il semble que le système de notification (notifyProcessFilter) n'existe plus.

Les plugins ont effectivement été renommées :
GenericPluginFilter => GenericAppImagePlugin
AreaPluginFilter => AreaAppImagePlugin
RectangularPluginFilter => RectangularAppImagePlugin

Concernant notifyProcessFilter,
C'est effectivement devenu notifyProgress(percent) ou notifyProgress(ProgressEvent).

J'ai oublié que j'avais changé ça. notifyProcessFilter était à la base une erreur d'écriture, j'avais toujours voulu écrire Progress

**pseudocode** · 07/01/2010, 14h01

Envoyé par Jack Sparrow

Les plugins ont effectivement été renommées :
GenericPluginFilter => GenericAppImagePlugin
AreaPluginFilter => AreaAppImagePlugin
RectangularPluginFilter => RectangularAppImagePlugin

Concernant notifyProcessFilter,
C'est effectivement devenu notifyProgress(percent) ou notifyProgress(ProgressEvent).

J'ai oublié que j'avais changé ça. notifyProcessFilter était à la base une erreur d'écriture, j'avais toujours voulu écrire Progress

Ok, ca compile mieux effectivement

J'ai modifié un filtre (bloom) pour utiliser le FastGaussianBlurPlugin. Pour l'instant je fais un appel direct des méthodes de l'instance. Je ne sais pas s'il faut passer par un Automate dans ce cas là.

Par contre il semble que le chargement/refresh dynamique de plugin ne fonctionne pas (menu "rafraichir plugins")

**Jack Sparrow** · 07/01/2010, 14h28

Envoyé par pseudocode

Ok, ca compile mieux effectivement

J'ai modifié un filtre (bloom) pour utiliser le FastGaussianBlurPlugin. Pour l'instant je fais un appel direct des méthodes de l'instance. Je ne sais pas s'il faut passer par un Automate dans ce cas là.

Par contre il semble que le chargement/refresh dynamique de plugin ne fonctionne pas (menu "rafraichir plugins")

Ce que je faisais pour les filtres souvent utilisés, je les cassais en 2 : Une partie plugin et une partie filter (BufferedImageOp) :

Exemple pour le ContrastPlugin :

http://subversion.developpez.com/pro...astPlugin.java
http://subversion.developpez.com/pro...tOperator.java

Mais il en reste probablement à couper.

Les automates permettent surtout d'enregistrer des séquences et de les réapliquer plus tard. Cela ressemble plus aux "scripts" et à la partie "automation" de photoshop.

**millie** · 07/01/2010, 18h10

Envoyé par pseudocode

- un zoom régulier avec la molette de la souris, pour que l'on se retrouve a 100% si on fait un zoom suivi d'un dézoom

Pour ça, j'ai juste remplacé ImagePanelServices#zoomService par :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
public void zoomService(int posx, int posy, boolean more) {
		double newZoom = panel.getImageInfo().getZoomFactor();
		
		if(more) {
			newZoom*=1.25;
		}
		else {
			newZoom*=0.8;
		}

Le zoom est un peu plus fort qu'avant par contre.

**pseudocode** · 07/01/2010, 18h55

Envoyé par millie

Pour ça, j'ai juste remplacé ImagePanelServices#zoomService par :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
public void zoomService(int posx, int posy, boolean more) {
		double newZoom = panel.getImageInfo().getZoomFactor();
		
		if(more) {
			newZoom*=1.25;
		}
		else {
			newZoom*=0.8;
		}

Le zoom est un peu plus fort qu'avant par contre.

pourquoi ne pas juste incrémenter/décrementer un entier, et utiliser une formule pour calculer le zoom factor ?

**millie** · 07/01/2010, 19h04

Envoyé par pseudocode

pourquoi ne pas juste incrémenter/décrementer un entier, et utiliser une formule pour calculer le zoom factor ?

Si on ne fait qu'incrémenter/décrémenter. Ca va faire un truc assez bizarre (ça dézoom très vite et ça zoome très lentement).

En fait, on pourrait simplement faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
private static final double zoomFactor = 0.9;
	
if(more) {
			newZoom*=1.0/zoomFactor;
		}
		else {
			newZoom*=zoomFactor;
		}

Mais il y a des micro décalages qui interviennent dû à des problèmes d'arrondi.
Maintenant, perso, j'en ai rien à carrer tellement c'est infime.

**pseudocode** · 07/01/2010, 19h18

Envoyé par millie

Si on ne fait qu'incrémenter/décrémenter. Ca va faire un truc assez bizarre (ça dézoom très vite et ça zoome très lentement).

Je vois pas en quoi ca serait différent d'avant ?

Code java :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
int zoom = 0;
 
void more() { zoom++; }
void less() { zoom--; }
 
double getZoomFactor() {
	if (zoom>0) return Math.pow(1.25,zoom);
	if (zoom<0) return Math.pow(0.8,-zoom);
	return 1;
}

Au pire, tu peux utiliser une table de lookup pour mettre les valeurs de zoomfactor que tu veux.

**millie** · 07/01/2010, 19h25

Ah oui, la fatigue

**pseudocode** · 07/01/2010, 19h48

Envoyé par millie

Ah oui, la fatigue

Vivement les vacances de Noël !

**millie** · 10/01/2010, 16h33

Envoyé par pseudocode

- un zoom par defaut a 100% si l'image tient dans la fenetre

Fait (et paramétrable dans les préférences).

Il reste le plus dur

**millie** · 10/01/2010, 16h47

Envoyé par pseudocode

Par contre il semble que le chargement/refresh dynamique de plugin ne fonctionne pas (menu "rafraichir plugins")

C'est corrigé. J'avais oublié une ligne de rafraîchissement

**millie** · 10/01/2010, 20h57

Envoyé par pseudocode

- un zoom de meilleur qualité (genre celui du plugin "redimensionnement texte"), car lors d'une réduction on perd des infos dans l'image, par exemple des morceaux de lignes fines

Bon, j'ai rapidement regardé pour mettre ça en place.
Avec une méthode simpliste, ça a tendance à mettre 0.2sec pour dézoomer sur une image de taille normale (1280*800).
Une technique serait de faire un premier rendu normal, de calculer un scaled instance dans un autre thread et de remettre à jour la vue...

Ca se fait, mais pas en 10min

. Normalement, les modifications ne seront localisées que dans la classe ImagePanel, donc ça va

**Jack Sparrow** · 11/01/2010, 12h56

Pour information, voici un workaround pour le Image#getScaledInstance avec un hint SMOOTH pour avoir quelque chose de plus rapide.

http://today.java.net/pub/a/today/20...dinstance.html

**pseudocode** · 11/01/2010, 14h09

Envoyé par Jack Sparrow

Pour information, voici un workaround pour le Image#getScaledInstance avec un hint SMOOTH pour avoir quelque chose de plus rapide.

http://today.java.net/pub/a/today/20...dinstance.html

Hum, vu le code de getScaledInstance(), je ne suis pas sur que ca change quoi que ce soit par rapport a un SCALE_AREA_AVERAGING

Code java :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
public Image getScaledInstance(int width, int height, int hints) {
        ImageFilter filter;
        if ((hints & (SCALE_SMOOTH | SCALE_AREA_AVERAGING)) != 0) {
            filter = new AreaAveragingScaleFilter(width, height);
        } else {
            filter = new ReplicateScaleFilter(width, height);
        }
        ImageProducer prod;
        prod = new FilteredImageSource(getSource(), filter);
        return Toolkit.getDefaultToolkit().createImage(prod);
    }

Par contre, d'après ton lien, l'interpolation bilineaire multi-step a l'air de donner des bons résultats.

EDIT: en fait non. Le multi-step est beaucoup plus long que le AreaAveraging.

**Jack Sparrow** · 11/01/2010, 14h28

Envoyé par pseudocode

Hum, vu le code de getScaledInstance(), je ne suis pas sur que ca change quoi que ce soit par rapport a un SCALE_AREA_AVERAGING

Je parlais effectivement du multi-step Bilinear.

D'après leur stats sur la réduction d'image (vu que c'est bien ça qui nous intéresse), ils donnaient 5ms pour le multi step et 31 pour l'area average.

**pseudocode** · 11/01/2010, 14h38

Envoyé par Jack Sparrow

Je parlais effectivement du multi-step Bilinear.

D'après leur stats sur la réduction d'image (vu que c'est bien ça qui nous intéresse), ils donnaient 5ms pour le multi step et 31 pour l'area average.

Les tests devaient être fait avec une ancienne implémentation de AreaAveragingScaleFilter car moi j'obtiens:

Image de départ 3543x2362 -> image d'arrivée 800x600
- SCALE_AREA_AVERAGING : 8.425003 ms
- MultiStep VALUE_INTERPOLATION_BILINEAR : 105.69857 ms

Je vais voir si on peut optimiser les choses pour le SCALE_AREA_AVERAGING dans notre ca, mais j'ai un doute.

[Suggestion] Un Zoom plus Zen

Millie

Discussions similaires

Partager

Partager