| sujet suivant »

[Transport Tycoon Forum Francophone]

Transport Tycoon Forum Francophone > OpenTTD > L'Artistik/Développement > Patch: Fenêtre d'observation

[[FR] Benoit]

Bonjour,

je viens de mettre au point mon premier patch avec interraction graphique. Il utilise la base du correctif "watch console". Cette commande additionnelle (made in Goulp) permet de centrer l'écran sur l'endroit où un joueur (au sens compagnie) effectue des actions de construction.

L'objectif premier de cette commande est de permettre à l'administrateur d'un serveur de vérifier si certains joueurs ne respectent pas les règles. Ensuite, ça permet aussi de regarder comment un joueur construit son réseau et éventuellement de le conseiller en direct.

L'inconvénient majeur est que l'on peut s'occuper que d'UN SEUL joueur. Après avoir publié sur tt-forum le code, un des contributeur a émit l'excellente idée d'utiliser les fenêtres de visualisation. Challenge très intéressant, dans la mesure où je n'avais jamais touché à la partie "GUI" d'openTTD.

C'est maintenant chose faite. Le correctif est en cours de test. Il a déjà fait l'objet d'une publication dans un ensemble de correctifs, et je pense que cette nouvelle version avec fenêtre va en intéresser plus d'un.

Maintenant, pour ceux que cela intéresse, il ne me reste plus qu'à mettre à disposition le code source, et un ensemble exécutable.

[06011988]

Intéressent comme patch

Jolie travail j'ai hate de voir tourner sa

[[FR] Benoit]

Maintenant, pour ceux que cela intéresse, il ne me reste plus qu'à mettre à disposition le code source, et un ensemble exécutable.

Le code source et la discute (en anglais dans le texte) sont disponibles sur tt-forums à l'adresse suivante : http://www.tt-forums.net/viewtopic.php?f=33&t=45221

Voici un aperçu de la fenêtre



Sur le coté gauche de la fenêtre, 15 boutons disponibles, qui s'activent en fonction du nombre de compagnies dans le jeu.
Il suffit d'appuyer sur le bouton dont la couleur correspond à la compagnie que l'on veut observer.
Il est possible d'ouvrir une autre fenêtre avec le 16ème bouton. ça permet d'observer plusieurs joueurs à la fois.
J'ai conservé les fonction Zoom In et Out de la fenêtre "Vue supplémentaire".

Pour l'ensemble exécutable, + traduction il faut que je mette à jour la version française, c'est un minimum, et aussi que je ponde une version linux. En effet, travaillant avec Visual Studio, je me suis limité, pour le moment, à la version windows.

Coté améliorations, évidemment, il y en a quelques-unes :

• Changer la couleur du fond de la barre de titre en fonction de la compagnie observée
• Afficher un indicateur d'activité par compagnie
• Pour les compagnies en multi-joueurs, pouvoir suivre 1 joueur au lieu de tous
• Activer la fenêtre d'observation depuis la fenêtre "Compagnie"

Et si vous avez d'autres idées, faites m'en part.

[[FR] Benoit]

Coté améliorations, évidemment, il y en a quelques-unes :

• Changer la couleur du fond de la barre de titre en fonction de la compagnie observée

Nouvelle version mise au point avec les améliorations citées avec en plus un bouton permettant de déplacer l'écran principal sur la vue de la fenêtre.

Aperçu de la nouvelle fenêtre



Edit 12:17

Je vous ai mis un exécutable + fichiers lng pour Windows à cet endroit : http://openttd.goulp.net/openttd_073_win32_watch_patch.zip

Comment l'utiliser, tout en gardant intact la version 0.7.3 ?

• Dupliquer le répertoire dans lequel se trouve la version 0.7.3
• Appeler ce répertoire "0.7.3_watch"
• Eventuellement copier le openttd.cfg (qui devrait se trouver dans le dossier "Mes Documents\openttd") dans ce nouveau répertoire "0.7.3_watch"
• Décompresser l'archive Zip dans ce répertoire en confirmant le remplacement des fichiers
• Démarrer l'openttd.exe
• Démarrer une partie local avec des AI activées ou connectez vous à une partie multi-joueurs (avec des joueurs connectés bien sur)
• Ouvrir la console (touche ²) ou menu ?/afficher console
• Taper "watch" et <Entrée>
• La fenêtre watch s'affiche, la console peut être fermée.
• Sélectionner la compagnie à suivre parmi les compagnies actives

Et bien sûr, faites un petit retour ici pour donner votre avis et les améliorations que l'on pourait y apporter !

[[FR] Benoit]

Coté améliorations, évidemment, il y en a quelques-unes :

• Changer la couleur du fond de la barre de titre en fonction de la compagnie observée
• Afficher un indicateur d'activité par compagnie

Et bien sur, un indicateur de présence pour chaque compagnie (voyant vert)
L'indicateur d'activité est en rouge.

Je n'avais pas pris le temps de mettre à jour par ici (alors que c'était fait sur tt-forums).

Donc version 3 disponible en téléchargement à cet endroit : http://openttd.goulp.net/openttd_073_win32_watch_patch_v3.zip

Même mode opératoire que précédemment.

Coté look, ça donne ceci (en mode opengfx)

[Skiper]

Wow, je viens de découvrir ça, c'est franchement une excellente idée!

Félicitations pour ton premier patch! 

[Belugas]

Mhhh... Rubidium, Yexo, Smatz qui participent...  Ok Ok, ca veux rien dire, mais c'est un quand même une bonne chose  
On peut toujours espérer pour toi  

Code:

+ /* Send information to the Watching windows */
+ // Search for all watching windows...

serait mieux :

Code:

+ /* Send information to the Watching windows
+ * Search for all watching windows... */

Et ailleurs aussi.  La regle : si c'est un commentaire écrit en bout de ligne, utilise "//"
Si le commentaire est seul sur une ligne (ou plusieurs), utilise "/* ... */"
Mais je vois que tu l'applique deja ici et la.  Un oubli passager, j'imagine...

Code:

+ if ( c->name != NULL ) {
+ SetDParamStr( 0, c->name );
+ } else {
+ SetDParamStr( 0, "unnamed" );

serait mieux ainsi :

Code:

+ SetDParamStr( 0, c->name != NULL ? c->name : "unnamed");

Mais... je me demande s'il ne serait pas plus "gracieux" d'utiliser le système de traductoin de l'application...
Beau travail

[[FR] Benoit]

Code:

+ if ( c->name != NULL ) {
+ SetDParamStr( 0, c->name );
+ } else {
+ SetDParamStr( 0, "unnamed" );

serait mieux ainsi :

Code:

+ SetDParamStr( 0, c->name != NULL ? c->name : "unnamed");

Mais... je me demande s'il ne serait pas plus "gracieux" d'utiliser le système de traductoin de l'application...

Beau travail

Merci pour le compliment ainsi que les conseils concernant les règles de codage. C'est en effet une étape importante avant de proposer le code pour une intégration dans le tronc.

Concernant l'utilisation des fichiers lang, j'ai déjà mis quelques affaires.

Par contre, le bout de code cité... n'est pas un bon exemple de ce qu'il faut faire....

je l'ai remplacé par le bout de code suivant (non publié encore...) :

Code:

if ( this->watched_company == INVALID_COMPANY || !IsValidCompanyID(this->watched_company) ) {
SetDParamStr( 0, "nothing" );
} else {
c = GetCompany( this->watched_company );
SetDParam(0, c->index);
GetString(company_name, STR_COMPANY_NAME, lastof(company_name));
SetDParamStr( 0, company_name );
}

Et puis il faut que je passe "nothing" dans la moulinette de traduction.

[[FR] Benoit]

Voici quelques nouvelles du Patch "Watch Company GUI"

La discussion autour de ce patch est toujours disponible sur tt-forums : http://www.tt-forums.net/viewtopic.php?f=33&t=45221

J'ai modifié le code pour les versions suivantes :

• 0.7.5 - mais non publié
• 1.0.0-beta3
• tronc r18972

avec les améliorations suivantes :

• Suppression de la commande console watch.
• Ajout d'un élément de menu "Watch Company" dans le bouton "Carte" de la barre d'outils.
• Utilisation de la couleur Orange lorsque la compagnie est une AI.
• Lors de l'activation pour une compagnie, déplace la fenêtre au dernier endroit construit.

Il faudrait que je mette à jour les sources qui fonctionnent pour la dernière version stable 0.7.5 afin que tout le monde puisse en profiter (récupérer, compiler, et s'amuser).

Par contre, si vous être intéressés par un package complet (exécutable + fichiers lang), faites moi signe et mettrai tout celà sur le site du Goulp avec les bonnes affaires (et la licence qui va bien - GNU Stuff).

Edit de 20:22

Un package <binaire Windows + lang + GNU> est dispnible ici : http://openttd.goulp.net/download/openttd_075_win32_watch_v6.zip

[Skiper]

J'aimerais bien essayer la nouvelle fonctionnalité. Mais je suis pas sûr de comment, je suis pas un fin technicien. 

[[FR] Benoit]

Même procédure que pour la 0.7.3 depuis ce message, légèrement modifiée pour la 0.7.5 !

Comment l'utiliser, tout en gardant intacte la version 0.7.5 ?

• Dupliquer le répertoire dans lequel se trouve la version 0.7.5
• Appeler ce répertoire "0.7.5_watch"
• Eventuellement copier le openttd.cfg (qui devrait se trouver dans le dossier "Mes Documents\openttd") dans ce nouveau répertoire "0.7.5_watch"
• Décompresser l'archive Zip dans ce répertoire en confirmant le remplacement des fichiers
• Démarrer l'openttd.exe
• Démarrer une partie local avec des AI activées ou connectez vous à une partie multi-joueurs (avec des joueurs connectés bien sur)
• Afficher la fenêtre à partir de la barre d'outils "Cartes", menu "Watch Company"
• Sélectionner la compagnie à suivre parmi les compagnies actives

Et bien sûr, faites un petit retour ici pour donner votre avis et les améliorations que l'on pourait y apporter !

J'aimerais bien essayer la nouvelle fonctionnalité. Mais je suis pas sûr de comment, je suis pas un fin technicien. 

Mais je viens de me souvenir que tu fonctionnes avec linux. Je n'ai pas, pour le moment, de binaire linux 0.7.5 modifié "watch company GUI".

[Skiper]

J'ai Wine, je peux essayer la version Windows que tu proposes.