Ajoute l'éligibilité au Livret d'épargne populaire

[guillett]

• Évolution du système socio-fiscal.
• Périodes concernées : toutes.
• Zones impactées : model/patrimoine/livret_epargne_populaire.py.
• Détails :
  • Ajoute l'éligibilité au Livret d'épargne populaire

---

[Morendil]

Yo @guillett le message TypeError que tu rencontres vient d'un conflit de nommage entre

• d'une part un des attributs de recarray, le type Numpy utilisé pour le "fancy indexing"
• d'autre part le nom d'un noeud paramètre.

En l'occurrence il s'agit de base. En renommant base en valeur_base (ou tout ce que tu veux qui n'est pas un attribut de recarray) le test passe. (Et ça peut sembler facile, dit comme ça, mais pour l'anecdote il y a plusieurs heures de debug pour en arriver là.)

La cause racine est le __getattr__ de VectorialParameterNodeAtInstant. En cherchant à bien faire (i.e. permettre une syntaxe condensée pour l'accès aux paramètres, à la parameters(period).epargne.livret_epargne_populaire) il me semble que ce code introduit de nombreuses chausse-trappes. On essaiera d'y prendre garde à l'avenir et peut-être serait-il utile de débarrasser cette zone (les classes Parameter etc.) de ces aspects "magiques".

Je t'ai aussi fait deux petites remarques. Mais à part ça merci pour cette contribution et surtout d'avoir pris le temps de la rendre plus élégante et lisible que la version de départ. :)

[guillett]

Franchement, 👏

(Et ça peut sembler facile, dit comme ça, mais pour l'anecdote il y a plusieurs heures de debug pour en arriver là.)

🙇‍♂️

[Morendil]

Il n'est plus nécessaire de poser tous les types énumérés dans model/base, le déplacer là ou la variable dont le type est cette Enum est déclarée.

[Morendil]

Mettre à jour label. :)

[Morendil]

(J'avais oublié de pousser la review avec les deux remarques mineures, désolé.)

[Morendil]

@guillett Si tu veux merger rapidement je peux te laisser le Changelog+setup et merge, sinon je devrais pouvoir faire ça d'ici demain.