fab-geocommuns / RNB-coeur

Le coeur du Référentiel National des Bâtiments : imports, APIs, logique métier
https://rnb.beta.gouv.fr
Apache License 2.0
3 stars 0 forks source link

Réparer les notebooks #455

Closed pauletienney closed 1 month ago

pauletienney commented 1 month ago

Quand je démarre les notebooks via docker exec -ti web python manage.py shell_plus --notebook, j'obtiens une erreur que je n'avais pas avant :

System check identified some issues:

WARNINGS:
rest_framework_tracking.APIRequestLog: (models.W042) Auto-created primary key used when not defining a primary key type, by default 'django.db.models.AutoField'.
    HINT: Configure the DEFAULT_AUTO_FIELD setting or the RestFrameworkTrackingConfig.default_auto_field attribute to point to a subclass of AutoField, e.g. 'django.db.models.BigAutoField'.
Traceback (most recent call last):
  File "/venv/lib/python3.12/site-packages/django_extensions/management/commands/shell_plus.py", line 281, in get_notebook
    from notebook.notebookapp import NotebookApp
ModuleNotFoundError: No module named 'notebook.notebookapp'

CommandError: Could not load shell runner: 'IPython Notebook'.

Cette erreur est décrite dans cette question SO. Cette PR downgrade la version de notebook en attendant qu'un fix soit apporté aux nouvelles versions.

Pour fonctionner, il est nécessaire de rebuild les containers docker.

fchabouis commented 1 month ago

il faudrait rajouter cette contrainte au niveau du fichier pyproject.toml (dans les dépendances de dev) qui est maintenant notre source de vérité pour les dépendances python

pauletienney commented 1 month ago

La version 6.5.6 de notebook est déjà intégrée à main. Plus besoin de cette PR.