Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gestion des équipes et de leur couleur #99

Closed
2 tasks
jenifaelle opened this issue Nov 1, 2016 · 2 comments
Closed
2 tasks

Gestion des équipes et de leur couleur #99

jenifaelle opened this issue Nov 1, 2016 · 2 comments
Assignees

Comments

@jenifaelle
Copy link
Contributor

jenifaelle commented Nov 1, 2016

Actuellement nous avons hardcode la valeur que nous sommes l'équipe bleu.
Cette considération devrait être gérée de façon plus générale

  • Modifiable par le UI
  • Modifiable par le referee (lorsqu'implémenté)

Il faudrait donc en faire la discussion d'architecture et minimiser les impacts.

@MaximeGLegault MaximeGLegault self-assigned this Nov 9, 2016
@MaximeGLegault
Copy link
Member

Les couleurs des équipes ne sont plus hardcodées. On peut passer un paramêtre ligne de commande pour changer la couleur des équipes lors du démarrage. On ne demande plus "notre_équipe_jaune?" c'est maintenant rendu "notre couleur == couleur x de l'enum couleur".
Pour modifier à partir de l'UI cela ne devrait pas être trop dur. Le framework est presque rendu à utiliser les packets de débogages, il ne lui reste qu'à répercuter le changement de couleur dans le reste du programme à voir...
Sinon, de l'arbitre est-ce que c'est necéssaire? Va-t-on ce faire demander de changer de couleur au milieu d'un match ou juste au début lors de notre initialisation? Le second cas est déjà couvert.

@TwistedSim
Copy link
Member

«Va-t-on ce faire demander de changer de couleur au milieu d'un match ou juste au début lors de notre initialisation? »
Il est possible de se faire demander de changer de couleur au milieu d'un match, mais on peut empêcher ca en donnant la couleur préférée à l'autre équipe...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants