Python3.7

[benjello]

• Changement mineur.
• Détails :
  • Migre de python3.6 à 3.7

---

• Jetez un coup d'œil au guide de contribution.
• Regardez s'il n'y a pas une proposition introduisant ces mêmes changements.
• [ ] Documentez votre contribution avec des références législatives.
• [ ] Mettez à jour ou ajoutez des tests correspondant à votre contribution.
• Augmentez le numéro de version dans setup.py.
• Mettez à jour le CHANGELOG.md.
• Assurez-vous de bien décrire votre contribution, comme indiqué ci-dessus

[sandcha]

P'tits soucis au make install :

ERROR: openfisca-survey-manager 0.38.0 has requirement numpy<1.16,>=1.11, but you'll have numpy 1.17.4 which is incompatible.
ERROR: flake8 3.6.0 has requirement pycodestyle<2.5.0,>=2.4.0, but you'll have pycodestyle 2.5.0 which is incompatible.

[benjello]

The first problem is linked with openfisca/openfisca-survey-manager#79
But we may release this constraint since now I am using virtualenv extensively.

[sandcha]

J'obtiens cette erreur au make test :

...
...openfisca-ceq/openfisca_ceq/tools/indirect_taxation/variables_generator.py", line 256, in generate_ad_valorem_tax_variables
    functions_by_name[func.__name__] = func
UnboundLocalError: local variable 'func' referenced before assignment

Il me semble qu'il manque quelques étapes de configuration pour l'exécution qui seront probablement aussi nécessaires à CircleCI.
L'erreur ci-dessus a été obtenue après :

1. make install
2. pip install .[ceq]
3. Dans ~/.config/openfisca-survey-manager/raw_data.ini, ajouté :

[ceq]
consumption_items_directory = .../ceq_consumption_items_directory

4. Dans le répertoire défini par consumption_items_directory, ajouté un fichier Produits_SEN.xlsx avec les colonnes suivantes :

code_coicop | label_variable | tva

mais aucune valeur.

@benjello Obtiens-tu la même erreur en local ?
Et, as-tu quelques informations à ajouter au fichier Produits_SEN.xlsx pour les tests ?

Mémo/À faire : vérifier les points listés ici tn#71

[sandcha]

Rendre cette dépendance à CEQ conditionnelle à un élément de configuration ?
Ici, on rend la dépendance obligatoire alors que nous avons un usage autre initié à l'OGP. 😕

[benjello]

J'arrive pas à rendre le truc optionnel et propre ...

[sandcha]

Et, si on conserve une dépendance à CEQ au chargement du modèle, s'assurer qu'il ne s'agisse pas d'un module tests ?

[sandcha]

Dispo pour aider là-dessus quand on aura des tests qui passent 🙂

[benjello]

Désolé @sandcha , il manque encore un fichier qui n'est pas complètement nettoyé.
Merci pour les rappels.