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

S'étendre à d'autres villes que Brest #7

Open
philae-ael opened this issue Feb 10, 2016 · 2 comments
Open

S'étendre à d'autres villes que Brest #7

philae-ael opened this issue Feb 10, 2016 · 2 comments
Milestone

Comments

@philae-ael
Copy link
Contributor

Reprise des message de #6
Benvi :

+1 et pourquoi pas en gros faudrait se baser sur "l'interface" de Adafruit_NeoPixel (qui n'est pas >énorme : https://github.com/jgarff/rpi_ws281x/blob/master/python/neopixel.py ).
Mais c'est à mon sens trop complexe pour le relier à des tests unitaires ... sauf si de l'autre coté tu
as aussi des jeux de données de test bibus en entrée :)
Donc se subtituer à l'API Bibus ce qui serait intéressant ça ouvrirait aussi la porte à d'autres API >(La Star à Rennes ...) ça nécessite de revoir le code et réfléchir à quelque chose d'ultra modulable >(regarder d'autres API).

Moi :

J'y avais pensé, mais j'avais pensé à juste créer un wrapper avec une api «de base» (d'après ce
que l'on utilise) qui sera redéfinie pour chaque api que l'on ajoute le problème c'est comment
passer d'une api à l'autre sans avoir à modifier le code (j'avais pensé à un __import__(api name)
mais je trouve ça moche)

Tristanplouz:

Pour développer le projet il faudrait qu'il soit construit en 3 partie:

  • Une partie API avec la gestion des API de Bibus, de la STAR, de KSMA,etc
  • Une partie qui traite les infos indépendantes des API et de l'affichage
  • Une partie qui gère l'affichage sur les LEDs ou sur un écran

Avec cette structure je pense que le projet sera entièrement modulable et ensuite plus simple à modifier

@philae-ael philae-ael added this to the Future milestone Feb 10, 2016
@Benvii
Copy link
Member

Benvii commented Feb 11, 2016

@naegi tu auras dans tous les cas besoin de faire un import :) après tu peux utiliser un patron de conception genre Adapter, tu as une implem pour chaque API qui est potentiellement complètement différente tu défini une interface et tu adapte chaque implem pour chaque API.

@philae-ael
Copy link
Contributor Author

C'est ce à quoi je pensais en parlant de créer un wrapper autour des api (faut vraiment que j'apprenne le nom de patrons de conception)

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

No branches or pull requests

2 participants