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

Mise à jour des communes associées à la mutuelle communale de la métropole de Montpellier #221

Merged
merged 8 commits into from
Jan 2, 2025

Conversation

@jenovateurs jenovateurs requested a review from Shamzic December 30, 2024 16:35
@jenovateurs
Copy link
Contributor Author

Merci pour tes suggestions d'amélioration @Shamzic. Par contre, pour le remplacement de sum par any, j'obtiens une erreur de mémoire. Je viens d'ajouter les modifs et attends les résultats de tests.

@Shamzic
Copy link
Contributor

Shamzic commented Jan 2, 2025

Tu peux tester localement avec la commande suivante sans attendre les tests via la CI :

openfisca_local_test tests/communes/montpellier/montpellier_mutuelle_communale.yml

Pour l'utilisation de sum, j'ai trouvé un exemple qui fonctionne ici :

def is_department_eligible(individu: Population, period: Period, parameters: ParameterNodeAtInstant) -> np.ndarray:
depcom = individu.menage('depcom', period)
eligible_departments = parameters.departements
eligibilities = np.array([
startswith(depcom, code.encode('UTF-8'))
for code in eligible_departments
])
return sum(eligibilities) > 0

Tu pourrais utiliser np.array(...) ou bien conserver ce que tu avais écrit tel quel en ajoutant > 0 à la fin du return

@jenovateurs jenovateurs merged commit 96fc838 into master Jan 2, 2025
11 checks passed
@jenovateurs jenovateurs deleted the montpellier-mutuelle-communal branch January 2, 2025 13:49
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