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

Ajout de la compatibilité numpy 1.25.0 #2124

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

eraviart
Copy link
Member

@eraviart eraviart commented Jun 19, 2023

  • Changement mineur.
  • Périodes concernées : toutes.
  • Zones impactées :
    • openfisca_france/model/prelevements_obligatoires/prelevements_sociaux/taxes_salaires_main_oeuvre.py
    • openfisca_france/model/prestations/minima_sociaux/asi_aspa.py
    • openfisca_france/model/prestations/cheque_energie.py
    • openfisca_france/model/prestations/locapass.py
  • Détails :
    • Enlève l'opérateur unaire "+" devant un booléen, car ce n'est plus supporté par Numpy 1.25

@eraviart eraviart marked this pull request as ready for review June 19, 2023 11:56
@eraviart eraviart requested review from benjello and sandcha June 19, 2023 11:56
@sandcha
Copy link
Contributor

sandcha commented Jun 19, 2023

Message de dépréciation initial : DeprecationWarning: Applying '+' to a non-numerical array is ill-defined. Returning a copy, but in the future this will error. (🔐 discussion slack)

Copy link
Contributor

@Allan-CodeWorks Allan-CodeWorks left a comment

Choose a reason for hiding this comment

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

Testé, tout est bon de mon côté.

Copy link
Contributor

@sandcha sandcha left a comment

Choose a reason for hiding this comment

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

Merci @eraviart pour ce correctif !

J'obtiens également la même liste d'expressions à mettre à jour (avec l'expression régulière [(]\n[ ]*[\+] dans VSCode).

Comment on lines +8 to +11
- `openfisca_france/model/prelevements_obligatoires/prelevements_sociaux/taxes_salaires_main_oeuvre.py`
- `openfisca_france/model/prestations/minima_sociaux/asi_aspa.py`
- `openfisca_france/model/prestations/cheque_energie.py`
- `openfisca_france/model/prestations/locapass.py`
Copy link
Contributor

@sandcha sandcha Jun 19, 2023

Choose a reason for hiding this comment

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

Suggested change
- `openfisca_france/model/prelevements_obligatoires/prelevements_sociaux/taxes_salaires_main_oeuvre.py`
- `openfisca_france/model/prestations/minima_sociaux/asi_aspa.py`
- `openfisca_france/model/prestations/cheque_energie.py`
- `openfisca_france/model/prestations/locapass.py`
- `model/prelevements_obligatoires/prelevements_sociaux/taxes_salaires_main_oeuvre.py`
- `model/prestations/minima_sociaux/asi_aspa.py`
- `model/prestations/cheque_energie.py`
- `model/prestations/locapass.py`

* Détails :
- Enlève l'opérateur unaire "+" devant un booléen, car ce n'est plus supporté par Numpy 1.25

## 149.1.0 [#2118](https://github.com/openfisca/openfisca-france/pull/2118)
Copy link
Contributor

Choose a reason for hiding this comment

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

🙏

@benjello
Copy link
Member

@eraviart @Allan-CodeWorks est-ce que les changements indiqués ici ont été intégrés ?

@Allan-CodeWorks
Copy link
Contributor

@eraviart @Allan-CodeWorks est-ce que les changements indiqués ici ont été intégrés ?

Non, mais il me semble qu'entre temps nous avons constaté des leaks lorsqu’on utilise numpy 1.25.0.
À ma connaissance on a downgrade vers la dernière version de numpy 1.24 et on s'est arrêté là.

Tant qu'on est pas en 1.25.0 il n'y a pas de problème 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants