missing dependency on pyyaml #10
Conversation
| self._actual_init() | ||
| return mtd(self, *args, **kwargs) | ||
| return initialized | ||
|
|
There was a problem hiding this comment.
Ce décorateur est-t-il utilisé pour des méthodes de classes définies dans bronx?
J'ai du mal à comprendre le besoin d'utiliser une autre fonction d'initialisation _actual_init plutot que __init__ -- ça me parait étrange. En tous cas, si ce décorateur reste dans bronx, il faudrait à mon avis que la docstring donne un exemple, ou a minima explique pourquoi il est utile.
There was a problem hiding this comment.
L'idée c'est de faire une "delayed init", en n'initialisant certaines choses (typiquement, lire un fichier de définitions GRIB) que lorsqu'on en a réellement besoin.
En fait je n'en ai réellement besoin que dans certaines classes d'epygram, donc je peux l'y garder, mais je pensais que ça pouvait être utile par ailleurs (et que la docstring était assez explicite).
| @@ -17,6 +17,7 @@ classifiers = [ | |||
| "License :: CeCILL-C Free Software License Agreement (CECILL-C)", | |||
| ] | |||
| dependencies = [ | |||
| "pyyaml", | |||
There was a problem hiding this comment.
De manière optionnelle oui. Mais tout code qui ferait appel à bronx pour ces fonctions matplotlib utiliserait déjà matplotlib, donc c'est moins indispensable.
Pour pyyaml c'est simplement qu'un
from bronx.datagrip.misc import read_dict_in_CSV
dans epygram lève une ModuleNotFoundError: No module named 'yaml'
And:
init_beforefrom epygram (used in other sub-projects)