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

Update notebooks to remove local data #30

Open
2 of 9 tasks
linogaliana opened this issue Aug 30, 2021 · 8 comments
Open
2 of 9 tasks

Update notebooks to remove local data #30

linogaliana opened this issue Aug 30, 2021 · 8 comments
Labels
help wanted Extra attention is needed

Comments

@linogaliana
Copy link
Contributor

linogaliana commented Aug 30, 2021

Notebooks here

  • examples/example_automotive_industry_sirene.ipynb
  • examples/example_gdp_growth_rate_yoy.ipynb
  • examples/example_insee_premises_sirene.ipynb
  • examples/example_doctors.ipynb
  • examples/example_population_pyramid.ipynb
  • examples/example_deaths_births.ipynb
  • examples/example_inflation_yoy.ipynb
  • examples/example_population_map.ipynb
  • examples/example_poverty_paris_urban_area.ipynb
@linogaliana
Copy link
Contributor Author

quarto should help solving this issue. When my PR will be solved (first #59), I might give it a try

@linogaliana
Copy link
Contributor Author

@linogaliana
Copy link
Contributor Author

@hadrilec je suis arrivé à une première version (en principe) fonctionnelle de :

qmd -> ipynb -> notebooks sauvés sur S3

Y a l'enjeu des runners qui font des timeout errors (débit réseau trop lent des runners de GHA) à gérer, j'y réfléchirai prochainement.

On est d'accord que c'est ce fichier (https://github.com/InseeFrLab/pynsee/blob/master/docs/examples.rst) qui gère la gallerie d'exemples sur le site readthedocs ? Y a un autre endroit où la localisation des notebooks intervient ?

@linogaliana
Copy link
Contributor Author

linogaliana commented May 12, 2022

Je pense que dans un premier temps le mieux est de laisser des notebooks sur le dépôt mais arrêter de les éditer pour privilégier l'édition des qmd. Cela évitera de casser la gallerie d'exemple sphinx.

Quand j'aurais le temps, je ferai un vrai site de showroom d'exemples où on pourra s'abstraire totalement des notebooks sur le dépôt.

Qu'en penses-tu @hadrilec ?

@hadrilec
Copy link
Contributor

hadrilec commented May 12, 2022

Salut @linogaliana, en effet c'est bien le fichier examples.rst qui définit la gallerie d'exemples.
Le fichier conf.py définit l'environnement du build de readthedocs. Lors du build le processus se place dans le dossier docs, et donc les chemins sont relatifs à ce placement.

Merci beaucoup pour les qmd! J'ai hâte de voir ce que ça va donner, à l'occasion ça m'intéresse de voir dans quel environnement tu testes ça?

OK ça marche, je vais arrêter de faire de nouveaux notebooks le temps que l'autre solution via quarto soit mise en place. Les dernières modifications que j'ai faites cette semaine correspondent à des bugs apparus suite à un changement de nom de paramètre. Pour l'instant je m'assure que les exemples tournent en les testant via un cron job quotidien: github/workflows/examples.yml, à l'avenir il faudra qu'on modifie légèrement ça pour faire tourner les qmd, a+

@linogaliana
Copy link
Contributor Author

Merci beaucoup pour les qmd! J'ai hâte de voir ce que ça va donner, à l'occasion ça m'intéresse de voir dans quel environnement tu testes ça?

Dans les services vscode du SSP-Cloud, quarto est pré-installé (j'ai une PR en attente sur le dépôt d'onyxia pour que les derniers réglages manquants soient ok)

OK ça marche, je vais arrêter de faire de nouveaux notebooks le temps que l'autre solution via quarto soit mise en place. Les dernières modifications que j'ai faites cette semaine correspondent à des bugs apparus suite à un changement de nom de paramètre. Pour l'instant je m'assure que les exemples tournent en les testant via un cron job quotidien: github/workflows/examples.yml, à l'avenir il faudra qu'on modifie légèrement ça pour faire tourner les qmd, a+

Merci je regarderai ce fichier

@RLesur
Copy link
Member

RLesur commented Feb 8, 2023

Hello @hadrilec , faut vraiment nettoyer ce repo qui devient trop gros et virer ces notebooks. Ca devient urgent, là...

@hadrilec
Copy link
Contributor

hadrilec commented Feb 8, 2023

Ok ça marche, @linogaliana si tu as un peu de temps pour me donner quelques conseils sur comment faire un site quarto qui pourrait accueillir la doc des fonctions et les exemples ça me ferait gagner un temps précieux

@hadrilec hadrilec added the help wanted Extra attention is needed label Feb 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants