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.

Sign up for GitHub

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

Jump to bottom

Met à jour la dépendance numpy à sa version 1.18 #1431

Merged
merged 3 commits into from
Dec 15, 2020
Merged

Met à jour la dépendance numpy à sa version 1.18 #1431

merged 3 commits into from
Dec 15, 2020

Conversation

sandcha
Copy link
Contributor

@sandcha sandcha commented May 9, 2020 •
edited
Loading

Connected to openfisca/openfisca-core#924

⚠️ Branche de référence sur CircleCI définie sur les travaux Core de mise à jour de numpy.

Documentation :

  • Amélioration technique non rétro-compatible.
  • Périodes concernées : toutes.
  • Détails :
    • Adapte OpenFisca-France à numpy v1.18 au travers de la mise à jour à Core v35.
    • Impacte les syntaxes des formules de calcul possibles du dépôt et de ses extensions.

Ces changements (effacez les lignes ne correspondant pas à votre cas) :

  • Modifient l'API publique d'OpenFisca France : par exemple renommage ou suppression de fonctions numpy employables dans les formules des variables du modèle et de ses extensions.

Quelques conseils à prendre en compte :

@sandcha
Copy link
Contributor Author

sandcha commented Oct 15, 2020

Rebase en cours.

@sandcha
Copy link
Contributor Author

sandcha commented Oct 15, 2020 •
edited by benjello
Loading

Occurrences de select à vérifier :

@benjello benjello mentioned this pull request Dec 3, 2020
Merged
@sandcha
Copy link
Contributor Author

sandcha commented Dec 3, 2020

Merci beaucoup @benjello pour toutes ces vérifications 🙌

En bilan : nous avons vérifié toutes les occurrences des éléments en fin de déprécation de la documentation numpy 1.18, que ce soit dans OpenFisca-Core et OpenFisca-France et avons amendé le code les rares fois où un cas déprécié était présent.

🤞Deux éléments n'étaient pas identifiables par chaîne de caractère simple :

  • Pour UFuncs with multiple outputs must use a tuple for the out kwarg., j'ai naïvement supposé que l'expression régulière ^[A-Za-z_]*, [A-Za-z_]* = suffirait à détecter les cas et n'en ai pas trouvé.
  • Pour Array order only accepts ‘C’, ‘F’, ‘A’, and ‘K’., j'ai supposé que cela n'affecterait que les ordonnancements explicites, à savoir, les numpy.array créés avec l'argument order=.

@sandcha
Copy link
Contributor Author

sandcha commented Dec 3, 2020

Rebase à venir. Celui-ci intègrera ces changements :

@sandcha
Copy link
Contributor Author

sandcha commented Dec 3, 2020

Rebase en cours.

@sandcha sandcha changed the title [WIP] Met à jour la version de la dépendance numpy à 1.18 Met à jour la dépendance numpy à sa version 1.18 Dec 3, 2020
sandcha
sandcha commented Dec 3, 2020
View reviewed changes
CHANGELOG.md Outdated
@@ -1,5 +1,11 @@
# Changelog

### ?

* Amélioration technique
Copy link
Contributor Author

@sandcha sandcha Dec 3, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dans le passé, un bump de numpy était considéré comme un patch.

Désormais, nous disposons d'extensions à OpenFisca-France pour lesquelles un bump patch de France lors du passage à numpy 1.18 pourrait générer des erreurs.
Je serais donc pour un changement de version majeur de France aux mises à jours intermédiaires de numpy.

Ceci ferait de la librairie numpy un cas particulier de versionnement pour openfisca.
Il me semble aussi que cela indiquerait que la syntaxe numpy fait partie du langage openfisca).

Qu'en dites-vous @benjello @maukoquiroga @guillett @frtomas @mtifarine ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je suis ok avec le changement de version majeure.

@sandcha sandcha marked this pull request as ready for review December 14, 2020 20:01
@sandcha
Copy link
Contributor Author

sandcha commented Dec 15, 2020 •
edited
Loading

Rebase et appel à Core v35, tel que nouvellement publié, en cours.

@sandcha sandcha merged commit 0490701 into master Dec 15, 2020
@sandcha sandcha deleted the bump-numpy branch December 15, 2020 15:00
@bonjourmauko bonjourmauko mentioned this pull request Mar 25, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benjello benjello benjello approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sandcha @benjello