Technical discussions | Discussions techniques
matttbe, Saturday 28 August 2010 à 17:03
|
|
Subscription date : 24 January 2009
Messages : 12573
|
Pour les merges, rien ne se fait par LaunchPad sauf la demande (et commentaires) mais je pense que ça peut se faire aussi depuis le terminal. En d'autres mots, il faut que quelqu'un lance bzr merge lp:~meuarrr/cairo-dock-core/SQP
bzr commit -m "Youhou, de nouvelles jauges! "
bzr push
Par contre, je ne vois pas pourquoi tu as ajouté "Cairo-Dock" pour approuver ce merge puisque c'est une fusion avec deux branches à toi (donc moi, je ne peux pas y toucher ).
Si la branche 'sqp' est la même que 'SQP' mais restée en version 651, tu peux donc supprimer 'sqp'. (Depuis https://code.launchpad.net/~meuarrr/cairo-dock-core/sqp).
Concernant le code, tout m'a l'air ok mais, sauf si fabounet est tenté, je pense qu'il vaut mieux respecter (par principe?) de ne pas ajouter de feature avant la sortie de la version stable même si le code est clair, propre et fonctionnel (même s'il n'affecte pas les autres jauges, etc. enfin, oui, ça pourrait être intégré dans trunk, je comprends...).
(Enfin, je ne pense pas que ça te bloque dans le développement... les branches sont là, en attente, rien n'est perdu!)
Concernant les "merge proposals", ce que tu peux faire, mais ce sera plus pour le fun , c'est de demander la fusion de la branche SQP avec lp:cairo-dock-core |
SQP, Sunday 29 August 2010 à 03:27
|
|
Subscription date : 03 July 2010
Messages : 1081
|
bon j'ai nettoyé ma branche pourrie (j'avais ajouté personne ca a du etre automatique).
Et ca fait plaisir d'avoir un premier avis sur le code
Je ferais la proposition de merge qd j'aurais eu au moins un premier testeur content.
Tout à fait ok pour pas pourrir la sortie d'une belle version très prometteuse qui devrait remporter un grand succès.
Pour la peine, je vous fais un "no comment" sur ma vidéo, et je vous laisse vous faire votre idée.
http://videobin.org/+1n4/1wg.html |
SQP, Sunday 29 August 2010 à 03:46
|
|
Subscription date : 03 July 2010
Messages : 1081
|
si je met quand même un commentaire, celui que j'ai préféré lacher dans le code (j'aurais pas aimé avoir à le recomprendre)
box positions :
*change axis to left border : -w / 2
*move to box #i : w * i / n
*move 1/2 box left : -w / 2n
=w(-0.5 +i/n -1/2n)
vertical mode is the same, but reverse (i => n-i-1) |
SQP, Monday 30 August 2010 à 04:12
|
|
Subscription date : 03 July 2010
Messages : 1081
|
bon alors je viens de voir passer un gros patch.
Si j'ai bien compris le code (j'ai parcouru en vitesse la partie renderer et gauge), on voit apparaitre la fonction rotate (pas encore maté sous quelle forme), des améliorations sur la partie affichage des texte des applets theme, et des options de thème pour le label.
Pour qqn qui ne voulait pas trop toucher le code avant la sortie, je trouve ca pas mal. Mais bravo ca a l'air sympa, et merci t'as écouté mes retours sur les problèmes liés à ca. Et même le prochain que j'allais exprimer, une fois debugué la taille de l'affichage du texte sur mon plugin, j'avais des trucs pourris au resize, et je voulais un 2e type de texte en taille fixe.
Alors voici mon premier bug, si j'affiche 2 valeurs, j'ai le texte 2x au centre. Ca tombe bien c'est pile ce sur quoi je bossais, je crois qu'il faudrait factoriser un peu de code au niveau du répartiteur de placement d'indicateurs pour pouvoir appliquer le texte au meme endroit.
Et ca serait bien en même temps de prévoir un moyen d'utiliser d'autres modes de placement : voir ma vidéo juste au dessus avec un mode shared space au lieu du gros + 3 petits autour.
Une petite clé dans le thème pour pouvoir choisir le type de placement serait pas mal.
|
Subscription date : 30 November 2007
Messages : 17118
|
J'ai enfin une vue sur la partie basse du graph qui est celle utilisée la majorité du temps
ah oui c'est nettement plus propre !
je pense qu'il vaut mieux respecter (par principe?) de ne pas ajouter de feature avant la sortie de la version stable
absolument, on n'est pas pressés non plus
j'ai fait qques commits tout de même, donc voila un bref résumé :
- ajout d'un booléen rotate_with_dock (pas testé, à voir son utilité)
- les jauges ne dessinent plus le texte (elles définissent juste l'emplacement des zones de texte)
- ajout de logos, donc là il y'a 3 types de zones:
- valeurs (formatées par l'applet avec la fonction format_value, du genre "5.2%" ou "150M", ne pas dépasser 5 caractères sinon c'est illisible)
- emblème (une petite image affichée par-dessus, pour l'instant aucun thème ne l'utilise)
- label ("CPU", "RAM", etc)
à chaque fois l'implémentation se contente de définir les positions et tailles des zones.
le dessin cairo est un peu à l'ouest, je fixe ça se soir. je revoit aussi un peu les thèmes des jauges pour leur mettre des zones (ce sera le répertoires "gauges2" sur le ftp)
ça n'est pas de grosses modifs en fait, y'a beaucoup de déplacement de code. j'ai pensé qu'il valait mieux le faire avant la sortie pour que les futurs thèmes soient compatibles. |
SQP, Monday 30 August 2010 à 19:08
|
|
Subscription date : 03 July 2010
Messages : 1081
|
au fait tu considères comme bon pour le test ou j'attend encore un peu avant de te sortir la liste ? (plus ma précédente remarque sur le placement des multi) |
fabounet, Tuesday 31 August 2010 à 12:45
|
|
Subscription date : 30 November 2007
Messages : 17118
|
pour le placement ça dépend fortement du rendu
par exemple un graphe va les positionner l'un en dessus de l'autre, mais peut aussi les dessiner sur un même graphe.
les barres, je pense que ce sera (c'est?) pareil.
mais pour les jauges qui sont plus carrées en général, l'empilement vertical n'est pas très optimisé.
bref, c'est assez délicat et j'ai laissé l'implémentation décider de faire comme elle veut
mais peut-être qu'effectivement on pourrait factoriser la chose. ça serait bien car ça éviterait de devoir calculer la position des zones de texte. |
fabounet, Wednesday 01 September 2010 à 14:16
|
|
Subscription date : 30 November 2007
Messages : 17118
|
bon j'ai fini mon nettoyage de printemps (un peu tardif )
donc voila ce qui a changé :
l'implémentation peut renseigner 3 types d'overlays : labels, valeurs, et emblèmes.
pour chaque il y'a la zone où ça s'affiche (position + taille) ainsi que couleur et alpha, en repère orthonormé centré [-0.5, +0.5].
si l'applet définit des labels/emblèmes ou demande d'afficher les valeurs sur l'icône, ces zones seront utilisées.
l'implémentation n'a pas a se soucier du chargement des surfaces/textures.
par contre, c'est finalement elle qui dessine les overlays (c'est plus pratique comme ça, et ça lui laisse plus de marge de manoeuvre), mais il y'a une fonction générique qui fait ça (cairo_dock_render_overlays_to_texture et cairo_dock_render_overlays_to_context).
si les valeurs sont écrites trop petites, le flag bCanRenderValueAsText est automatiquement désactivé et les valeurs sont écrites en quick-info.
s'il y'a besoin de factoriser des choses entre plusieurs implémentations, il vaut mieux le mettre dans dossier implementations.
voila, maintenant je pense que tu peux développer ton rendu "barre de progression" en tout sérénité |
SQP, Wednesday 01 September 2010 à 18:11
|
|
Subscription date : 03 July 2010
Messages : 1081
|
voila, maintenant je pense que tu peux développer ton rendu "barre de progression" en tout sérénité
j'ai pas mal de tests à faire sur tes modifs pour voir ce que ca donne (surtout pour les textes), mais mon premier patch ne devrait pas avoir changé.
les seuls problèmes qui se posaient vraiment étaient les problèmes génériques aux renderers pour les textes et aux thèmes qui sont tous en ratio fixe pour le moment.
Il faut que je vérifie si le fallback vers le quickinfo règle les problèmes que j'avais repéré (jusqu'ici il était pourri si on avait réduit de moitié la hauteur de l'icone).
Mais pour mon utilisation perso, un taille fixe devrait etre nickel (clé label si j'ai bien vu).
Donc je vais reapply mon patch effect, et combiner mes tests de texte avec.
petite question annexe. J'ai fait ma branche sur ta version 650. Comment je fais pour backporter tes modifs dedans ? (je me met à bzr 8-)) (et on peut faire sauter completement un patch d'un coup ?)
mais avant, en 655 (j'étais sur ta 653 je crois avant), si j'active une gauge (sysmon ou netspeed) j'ai ca x10 environ des qu'il doit afficher : cairo_dock_render_gauge_opengl: assertion `pGauge != NULL && pGauge->pIndicatorList != NULL' failed |
matttbe, Wednesday 01 September 2010 à 21:31
|
|
Subscription date : 24 January 2009
Messages : 12573
|
J'ai fait ma branche sur ta version 650. Comment je fais pour backporter tes modifs dedans ? Tu mets à jour ta branche (pull) et tu push dessus
et on peut faire sauter completement un patch d'un coup ? bzr revert?
PS: n'oublie pas de structurer tes commentaires avec des points |
SQP, Wednesday 01 September 2010 à 21:42
|
|
Subscription date : 03 July 2010
Messages : 1081
|
PS: n'oublie pas de structurer tes commentaires avec des points
c'est le défaut quand on en met
Pour le bzr, faut que je prenne le temps de m'amuser avec oui. Mais la j'avais plusieurs raisons pour revenir à l'originale (aider au debug principalement) donc j'ai laissé de coté ma branche pour le moment. Et vu ce qu'il y a dessus, je suis aussi bien de la casser et remettre à la main mon unique patch encore valable (et ca me permettra de l'améliorer un peu avec un struct pour le type d'effet). |
matttbe, Thursday 02 September 2010 à 00:50
|
|
Subscription date : 24 January 2009
Messages : 12573
|
Le plus simple avec BZR est peut-être de dupliquer les dossiers de tes branches trunk et d'y faire les modifs à côté. |
fabounet, Thursday 02 September 2010 à 09:53
|
|
SQP, Thursday 02 September 2010 à 10:14
|
|
Subscription date : 03 July 2010
Messages : 1081
|
tu peux pas juste revert le gauges2 le temps de finir qu'on évite de multiplier les rapports ?
http://www.glx-dock.org/bg_topic.php?t=4740&pos=0#mess_61323
changer 4 caractères corrige les 2 bugs
et on est au moins 5 à l'avoir chopé dejà je crois
Edit : note pour matttbe qui voulait faire des paquets ubuntu hier : c'est pas le bon moment je crois |
fabounet, Thursday 02 September 2010 à 10:31
|
|
Subscription date : 30 November 2007
Messages : 17118
|
je les remets ce soir
je voulais le faire hier mais j'ai eu du mal sur un bug de Lylambda |
SQP, Thursday 02 September 2010 à 10:47
|
|
Subscription date : 03 July 2010
Messages : 1081
|
perso c'est bon j'ai corrigé, je peux travailler tranquillement
enfin je garde comme prio aujourd'hui les menu applets, mais je te ferais un petit rapport sur mes premiers tests de theme |
matttbe, Thursday 02 September 2010 à 10:50
|
|
Subscription date : 24 January 2009
Messages : 12573
|
@ Fabounet, va falloir se depecher avec la release sinon on va etre perdu avec les posts de SQP et il risque d'exploser |
fabounet, Thursday 02 September 2010 à 11:13
|
|
Subscription date : 30 November 2007
Messages : 17118
|
oui oui, je tente de passer sur IRC ce soir (il me reste plus que les jauges à vérifier, ça ne prend pas trop d'occupation cerveau ) |
SQP, Thursday 02 September 2010 à 11:16
|
|
Subscription date : 03 July 2010
Messages : 1081
|
pas trop vite quand meme, ca prend du temps de convaincre fab de rentrer un patch core, et il m'en faut encore un peu pour faire passer le patch effects (tellement insignifiant par rapport à tout se qui se trame autour de lui).
Mais t'inquiete pas pour moi, j'ai volontairement limité le nombre de sujets que j'ai ouvert, pour pas trop vous noyer dans mon flood et pour en garder pour les prochains mois.
Mais après c'est plus du détail et du cosmétique.
par exemple, j'aimerais bien que les applets qui proposent plusieurs actions proposent une select pour choisir celle associée aux differentes actions utilisateur
(ex pour pouvoir choisir de lancer un xkill sur le clic milieu sur le sysmon ou si je préfère un expose sur le middle clic sur switcher) |
lylambda, Thursday 02 September 2010 à 13:46
|
|
Technical discussions | Discussions techniques
|