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

Ajout de la plateforme Clikodoc #530

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Conversation

Bouska
Copy link
Contributor

@Bouska Bouska commented May 30, 2021

** Description **
Ma première ébauche pour l'ajout de la plateforme Clikodoc. Il a encore un certain nombre de choses qui ne fonctionnent pas, du code de debug qui traîne, et des tests inexistants à rajouter. Niveau architecture, c'est un mix de ce qui a été fait sur Doctolib et ordoclic, plus ma sauce maison.

** TODO list **

  • Faire fonctionner le code
  • Enlever les codes de debug and co
  • Ajouter des tests unitaires

Pablo Martin-Gomez added 4 commits May 29, 2021 13:26
Some centers have multiples pratices in the same location. With cache, we avoid having to call multiple times the API with the same address.
@Bouska
Copy link
Contributor Author

Bouska commented Jun 5, 2021

J'ai enfin un code fonctionnel, mais j'ai deux soucis :

  • c'est extrêmement lent, un centre avec des dispo prend jusqu'à 1m30 (heureusement qu'il n'y en a que 2)
  • j'ai besoin d'un token pour faire des appels d'API, que je récupère avec un GET sur la homepage ; avec le multiprocessing, je me retrouve à faire 35 calls utiles et 33 calls pour récupérer un token, alors que je pourrais tout à fait partager un token pour tous les appels, il faut que je regarde comment améliorer ça

@Bouska
Copy link
Contributor Author

Bouska commented Jun 6, 2021

* j'ai besoin d'un token pour faire des appels d'API, que je récupère avec un GET sur la homepage ; avec le multiprocessing, je me retrouve à faire 35 calls utiles et 33 calls pour récupérer un token, alors que je pourrais tout à fait partager un token pour tous les appels, il faut que je regarde comment améliorer ça

Oubliez ce que j'ai dit, de un je ne sas pas compter et de deux je ne sais pas faire une comparaison, maintenant je n'ai que 3 calls pour récupérer un token et j'ai gagné 15sec par centre avec des créneaux 🚀 😅

@Bouska
Copy link
Contributor Author

Bouska commented Jun 6, 2021

Après moultes itérations, j'ai fait passé mon pire centre de 1m30 à 12sec, je pense qu'on ne fera pas mieux. Plus qu'à nettoyer le code et implémenter mes tests \o\

@fcamblor
Copy link

Hi ✋

Des news sur l'inté de cette plateforme ? :-)
(entretemps, la révolution des créneaux quotidiens est apparue, je pense qu'il faudra procéder à quelques ajustements...)

@Bouska
Copy link
Contributor Author

Bouska commented Aug 22, 2021

Des news sur l'inté de cette plateforme ? :-)
(entretemps, la révolution des créneaux quotidiens est apparue, je pense qu'il faudra procéder à quelques ajustements...)

Je n'ai pas vraiment eu de temps à y consacrer et je ne pense pas que j'en aurais tout de suite. Si quelqu'un veut reprendre le sujet, vous êtes le bienvenue. Il y a effectivement "la révolution des créneaux quotidiens" à gérer mais de la manière dont j'ai codé le bazar les informations sont déjà récupérées et stockées (temporairement), il n'y a qu'à faire remonter les infos, probablement trois quatre lignes de debug à enlever et des test à rajouter (:cold_sweat:)

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

Successfully merging this pull request may close these issues.

2 participants