Feature/age gap analysis #23
Conversation
| @@ -5,8 +5,8 @@ | |||
| from bs4 import BeautifulSoup | |||
| import re | |||
| import outputformat as ouf | |||
| import wikipediaapi | |||
| from bechdelai.data.scrap import get_json_from_url | |||
There was a problem hiding this comment.
Utilise plutôt ici from .scrap import get_json_from_url en import relatif, ça permet d'éviter des bugs
| @@ -0,0 +1,87 @@ | |||
| import sys | |||
There was a problem hiding this comment.
Qu'est-ce que fait ce script ? Il permet d'aller sauvegarder des données ?
Est-ce qu'il faut le mettre dans la librairie ?
| @@ -0,0 +1,99 @@ | |||
| import sys | |||
There was a problem hiding this comment.
On peut soit garder pour le moment toute la démo streamlit dans les notebooks, soit le sortir dans un autre repo (peut être dans un second temps)
Si des scripts sont importants -> les mettre dans la librairie
There was a problem hiding this comment.
En vrai pas besoin de changer grand chose :
Il y a 3 catégories dans ce groupe de commits :
- La démo sur streamlit
- Des données précalculées (CSV ou graphe en HTML)
- Les utilitaires sur l'âge
On pourrait garder pour le moment la démo streamlit en explo dans le dossier notebook, avec l'objectif de le sortir du répo pour la mettre en ligne sur Hugging Face Spaces. Mais pour le moment pas grave.
Important de sortir par contre dans la librairie toutes les fonctions qui pourraient être utilisées en dehors de la démonstration (soit les fichiers complets, soit des bouts de la démo) quitte à refactorer la démo derrière.
Attention j'ai l'impression qu'il va y avoir un conflit sur le pyproject.toml aussi comme ça fait longtemps
| @@ -0,0 +1,373 @@ | |||
| import sys | |||
There was a problem hiding this comment.
Ce script là pourrait être dans la librairie parce qu'on pourrait l'utiliser dans le notebook comme dans la démo
No description provided.