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

Préfère l'utilisation explicite du namespace numpy #1398

Closed
wants to merge 22 commits into from

Conversation

bonjourmauko
Copy link
Member

@bonjourmauko bonjourmauko commented Jan 5, 2020

See openfisca/openfisca-core#673
See #1395 (comment)

  • Amélioration technique.
  • Périodes concernées : toutes.
  • Zones impactées : **/*.
  • Détails :
    • Préfère l'utilisation explicite du namespace numpy.
    • L'utilisation des namespaces est une bonne pratique Python

Ces changements :

  • Corrigent ou améliorent un calcul déjà existant.

Quelques conseils à prendre en compte :

@benjello
Copy link
Member

benjello commented Jan 5, 2020

Tout compte fait je ne sais pas si c'est une bonne idée de changer cela. Cela rend la lecture des formule plus compliquée pour les économistes (et pour tout un chacun) pour éviter que les gens oublie qu'openfisca travaille sur des vecteurs ! Surtout que l'on parle d'un paquet pays.

Certes il y a eu quelque cas de méprise mais je crains que l'on ne crée plus de désagréments que l'on en résout.

@bonjourmauko bonjourmauko changed the title [WIP] Use import numpy Préfère l'utilisation explicite du namespace numpy Jan 5, 2020
@bonjourmauko bonjourmauko requested review from Morendil and a team January 5, 2020 21:18
@bonjourmauko bonjourmauko requested a review from benjello January 5, 2020 21:20
@bonjourmauko
Copy link
Member Author

Merci @benjello j'ai ajouté « need consensus » à la pull request.

@bonjourmauko
Copy link
Member Author

@benjello @guillett Une solution moins intrusive pour arriver au même objectif serait d'être explicite sur le contrat des formules, not. sur le return (qu'invariablement est un ndarray).

Cela vous semble-t-il un bon compromis ? Sinon, avez-vous d'autres idées pour arriver au même objectif ?

Je crois que c'est une bonne direction à suivre, or je suis d'accord qu'il faut mesurer l'impact du changement sur les rédacteurs et trices de formules.

cc @Morendil @sandcha

@benjello
Copy link
Member

benjello commented Jan 9, 2020

Je comprends l'intérêt mais je suis à peu près sûr que cela va parasiter les économistes.
Je suis pour éviter toute indication de typage dans les paquets pays.
Idéalement on voudrait être purement déclaratif pour les formules mais comme on veut avoir la flexibilité de python on a une solution bâtarde mais qui me semble un bon compromis.

@bonjourmauko
Copy link
Member Author

Je ferme pour le moment car pas de consensus sur le sujet.

@bonjourmauko bonjourmauko deleted the use-import-numpy branch October 18, 2020 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants