Skip to content

Conversation

benjello
Copy link
Member

@benjello benjello commented Oct 1, 2025

Technical changes

  • Fix vectorial parameter acces with integer key

Copy link
Member Author

@benjello benjello left a comment

Choose a reason for hiding this comment

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

@guillett
Copy link
Member

guillett commented Oct 1, 2025

Oupsyyy
Merci @benjello pourrais-tu ajouter un test ?

@guillett
Copy link
Member

guillett commented Oct 1, 2025

Et désolé @clallemand

@guillett
Copy link
Member

guillett commented Oct 8, 2025

Je ne crois pas que ça soit lié à ça cf. openfisca/openfisca-france#2589

J'ai tenté de faire un test en plus de tests/core/parameters_fancy_indexing/test_fancy_indexing.py:test_with_properties_starting_by_number mais sans succès.

Étant donné que ça n'a pas l'air d'être ça le problème, je ne regarde pas plus pour le moment.
Pour moi, on peut faire cette PR.

@guillett
Copy link
Member

guillett commented Oct 8, 2025

Désolé, en fait, dans un premier temps, je n'avais vu qu'une partie des tests en échec.

Je viens de corriger cette PR en ajoutant un test.

J'ai lancé les tests mais il reste encore des cas à traiter côté France et CSS.

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.

2 participants