-
Notifications
You must be signed in to change notification settings - Fork 82
Fix vectorial parameter access with integer keys #1336
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Related to openfisca/openfisca-france#2587
cc @clallemand
Oupsyyy |
Et désolé @clallemand |
Je ne crois pas que ça soit lié à ça cf. openfisca/openfisca-france#2589 J'ai tenté de faire un test en plus de Étant donné que ça n'a pas l'air d'être ça le problème, je ne regarde pas plus pour le moment. |
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. |
@clallemand est-ce que l'on merge et on refera des tests sur france plus tard ? |
Non non, ça passe pas encore sur France. Je vais avancer là dessus rapidement. J'ai pas encore des tests sur core mais je vois ceux qui échouent sur France. |
Bonjour, merci de gérer cette correction. Je devrais être davantage réactive s'il faut tester des choses maintenant ! |
Technical changes