search

legras / uav3i

HMI for Interreg IV A program 3i
0 stars 1 forks source link

Question #36

Closed phtanguy closed 10 years ago

phtanguy commented 10 years ago

Test inutile ?

Dans BoxMnvr (lignes 197 à 201), les tests suivants sont utiles ?

                if (_isNorthSouth && boxhndl.isNorthsouth())
                    length = max/2;

                if (!_isNorthSouth && !boxhndl.isNorthsouth())
                    length = max/2;

length est toujours égal à max/2.

legras commented 10 years ago

Pas regardé le reste du code, mais si _isNorthSouth == true et boxhndl.isNorthsouth() == false, on ne touche pas à la valeur de length.

En gros, c’est un NOT XOR, il y a sûrement plus élégant pour l’écrire.

Le 7 juil. 2014 à 11:06, Philippe TANGUY notifications@github.com a écrit :

Test inutile ?

Dans BoxMnvr (lignes 197 à 201), les tests suivants sont utiles ?

            if (_isNorthSouth && boxhndl.isNorthsouth())
                length = max/2;

            if (!_isNorthSouth && !boxhndl.isNorthsouth())
                length = max/2;

length est toujours égal à max/2.

— Reply to this email directly or view it on GitHub.

phtanguy commented 10 years ago

J'ai dû n’emmêler les pinceaux... je pensais que dans ce cas length ne pouvait qu'être égal à 0 et ce n'est pas le cas.

Une autre question

J'aurais dû mettre au pluriel le titre du ticket...

Dans SymbolMap, le dessin du drone est fait deux fois : lignes 264 à 289 et une seconde fois juste après (lignes 292 à 310). Tu vois une explication ? Pour le moment, j'ai commenté le second dessin et ça fonctionne (commit e74c2dc403935a88b1ef755ae7d3175f95c43683).

legras commented 10 years ago

Démêle tes pinceaux Philippe, le deuxième dessin c’est pour le REPLAY.

On continue d’afficher en gris la position actuelle, et on affiche en rouge la position passée.

Le 7 juil. 2014 à 11:52, Philippe TANGUY notifications@github.com a écrit :

J'ai dû n’emmêler les pinceaux... je pensais que dans ce cas length ne pouvait qu'être égal à 0 et ce n'est pas le cas.

Une autre question

J'aurais dû mettre au pluriel le titre du ticket...

Dans SymbolMap, le dessin du drone est fait deux fois : lignes 264 à 289 et une seconde fois juste après (lignes 292 à 310). Tu vois une explication ? Pour le moment, j'ai commenté le second dessin et ça fonctionne (commit e74c2dc).

— Reply to this email directly or view it on GitHub.

phtanguy commented 10 years ago

Bon... Ne dis à personne que je suis con ! :) Je n'avais pas fait attention au if (MainFrame.SWITCHER.getMode() == Switcher3Buttons.Switcher3ButtonsMode.REPLAY)

matSimonnet commented 10 years ago

Promis ! Si on me demande je dis que je ne suis pas au courant ;-)

On Mon, Jul 7, 2014 at 1:21 PM, Philippe TANGUY notifications@github.com wrote:

Bon... Ne dis à personne que je suis con ! :) Je n'avais pas fait attention au if (MainFrame.SWITCHER.getMode() == Switcher3Buttons.Switcher3ButtonsMode.REPLAY)

Reply to this email directly or view it on GitHub https://github.com/legras/uav3i/issues/36#issuecomment-48166754.

phtanguy commented 10 years ago

Et ben voilà, ça commence à se savoir...

On 7 juillet 2014 13:39:22 CEST, Mathieu Simonnet notifications@github.com wrote:

Promis ! Si on me demande je dis que je ne suis pas au courant ;-)

On Mon, Jul 7, 2014 at 1:21 PM, Philippe TANGUY notifications@github.com wrote:

Bon... Ne dis à personne que je suis con ! :) Je n'avais pas fait attention au if (MainFrame.SWITCHER.getMode() == Switcher3Buttons.Switcher3ButtonsMode.REPLAY)

Reply to this email directly or view it on GitHub https://github.com/legras/uav3i/issues/36#issuecomment-48166754.

Reply to this email directly or view it on GitHub: https://github.com/legras/uav3i/issues/36#issuecomment-48168063

Envoyé de mon téléphone Android avec K-9 Mail. Excusez la brièveté.

phtanguy commented 10 years ago

Encore une question/remarque

Pour le moment, la liste des manœuvres qui ont été dessinées sur la table sont présentes dans la classe SymbolMap (c'est de la vue). Cependant comme maintenant on transmet une manœuvre sur le Veto, que l'opérateur accepte ou pas, ça devient aussi du modèle. De plus, d'un point de vue pratique, ce serait pour moi plus facile d'y avoir accès via la classe UAVModel. Pour le moment, j'ai fait crado, j'ai simplement fait un getter static sur _SMAP dans SymbolMap afin de retrouver la manœuvre correspondant à la demande pour pour pouvoir changer son état.

--> Petite erreur : le getSymbolMap() est fait dans MainFrame puis je fais une recherche pour trouver la manoeuvre correspondante à mettre à jour.

On en rediscute.